Migration url fixing
AbandonedPublic

Authored by a2batic on Feb 13 2017, 1:15 PM.

Details

Reviewers
jskladan
Group Reviewers
resultsdb
Summary

Migration URL

Test Plan

NA

Diff Detail

Repository
rRSAPI resultsdb_api
Branch
feature/fixurl (branched from develop)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 1049
Build 1049: arc lint + arc unit
a2batic retitled this revision from to Migration url fixing.Feb 13 2017, 1:15 PM
a2batic updated this object.
a2batic edited the test plan for this revision. (Show Details)
a2batic added a reviewer: resultsdb.
jskladan requested changes to this revision.Feb 13 2017, 1:20 PM
jskladan added a reviewer: jskladan.
jskladan added a subscriber: jskladan.
jskladan added inline comments.
setup.py
28

The project, in fact, lives on pagure now - we just recently moved it, so if you want to change it, might make more sense to use the current value.

This revision now requires changes to proceed.Feb 13 2017, 1:20 PM

You could have updated this revision, instead of creating D1130, but I guess Phabricator is a bit complicated for the first time user.
Please abandon this revision (go down to "add comment section" and select "abandon" in the "action" field), so we don't have unnecessary duplicate.
Thanks!

Well, before you do what @jskladan told you, let me ask you a question :) Do you wish to become familiar with sending patches for Taskotron projects in Phabricator? Because there's some non-trivial knowledge and work required - Phabricator workflow is a bit different from github/pagure workflow, and some Taskotron projects require multiple setup steps (creating a virtualenv, so that test suite passes, etc). @sumantrom expressed the interested to learn the workflow, so we're teaching him. But if you're only interested in extremely tiny changes like fixing a hyperlink once a year, we don't really want to be hard on you and force you to learn the full workflow :) So if you don't think you'll be sending patches into Phab regularly, and you'd rather spend your time on different tasks, just tell us, and we'll simply commit the change right away. Also, in that case, next time you don't even need to bother with creating a diff, just report a new issue and tell us that such and such link is outdated. That's very quick to do, no deep Phabricator/Taskotron knowledge needed.

So it's really up to you, what do you want to do and how. Feel free to tell us.

@jskladan @kparal, Thanks for helping out and asking. @sumantrom helped me with this diff. I am not sure if I will be sending many diffs for Taskotron projects for now, but yes, I am up for learning :)

setup.py
28

Updated! Created new diff here

Great. Now let's abandon this revision in favor of D1130. Please select "Abandon revision" from the Action picker. Thanks.

a2batic abandoned this revision.Feb 14 2017, 9:55 AM

Thanks!