Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed Buildbot Categories #419

Merged
merged 1 commit into from Jul 1, 2016

Conversation

@this-jacob
Copy link
Contributor

this-jacob commented Jun 30, 2016

Addresses Issue: #412

Removed unused categories on the builders.

Additionally the categories parameter was deprecated in Buildbot 0.8.12 (the version currently in use)


This change is Reviewable

@aneeshusa
Copy link
Member

aneeshusa commented Jul 1, 2016

Thanks for the PR! The code changes look good but I have a few comments about the commit message:

  • Please use imperative tense and start with a capital letter. See this example git commit message style guide.
  • It would be nice to add a few lines of description in the commit message body about why we are making this change (e.g., this has been deprecated and we don't use it anyways).
@this-jacob this-jacob force-pushed the this-jacob:remove_buildbot_categories branch from dc0f6ce to bc00b2d Jul 1, 2016
@this-jacob
Copy link
Contributor Author

this-jacob commented Jul 1, 2016

@aneeshusa Do these changes work?

@this-jacob this-jacob force-pushed the this-jacob:remove_buildbot_categories branch from bc00b2d to 4c4c19f Jul 1, 2016
@aneeshusa
Copy link
Member

aneeshusa commented Jul 1, 2016

Just a few spelling/grammar nits:

  • paramater -> parameter
  • depricated -> deprecated
  • the parameters were removed -> remove it. (with the period at the end)
Jacob Thomas
Since the categories parameter was deprecated in Buildbot version
0.8.12, the version currently in use, remove it.
@this-jacob this-jacob force-pushed the this-jacob:remove_buildbot_categories branch from 4c4c19f to 987b364 Jul 1, 2016
@this-jacob
Copy link
Contributor Author

this-jacob commented Jul 1, 2016

@aneeshusa okay, I fixed those spelling issues

@aneeshusa
Copy link
Member

aneeshusa commented Jul 1, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Jul 1, 2016

📌 Commit 987b364 has been approved by aneeshusa

@bors-servo
Copy link
Contributor

bors-servo commented Jul 1, 2016

Testing commit 987b364 with merge 351fc5d...

bors-servo added a commit that referenced this pull request Jul 1, 2016
Removed Buildbot Categories

Addresses Issue: #412

Removed unused categories on the builders.

Additionally  the `categories` parameter was deprecated in Buildbot 0.8.12 (the version currently in use)

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/419)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 1, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit 987b364 into servo:master Jul 1, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.