Skip to content

diagnostic: remove duplicate checks.#899

Merged
MikeMcQuaid merged 1 commit intoHomebrew:masterfrom
MikeMcQuaid:diagnostic-duplicate-remove
Sep 9, 2016
Merged

diagnostic: remove duplicate checks.#899
MikeMcQuaid merged 1 commit intoHomebrew:masterfrom
MikeMcQuaid:diagnostic-duplicate-remove

Conversation

@MikeMcQuaid
Copy link
Copy Markdown
Member

@MikeMcQuaid MikeMcQuaid commented Sep 9, 2016

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes? Here's an example.
  • Have you successfully run brew tests with your changes locally?

Remove duplicate checks added in #896.

@BrewTestBot BrewTestBot added the in progress Maintainers are working on this label Sep 9, 2016
@ilovezfs
Copy link
Copy Markdown
Contributor

ilovezfs commented Sep 9, 2016

may have to disable that "Required" thing ... I guess something in test-bot is out of order so it's still using the old bits?

@MikeMcQuaid
Copy link
Copy Markdown
Member Author

@ilovezfs Nope, valid test failure 😀

@ilovezfs
Copy link
Copy Markdown
Contributor

ilovezfs commented Sep 9, 2016

ah because you didn't update the tests that go with the diagnostic changes... I see

@ilovezfs
Copy link
Copy Markdown
Contributor

ilovezfs commented Sep 9, 2016

💚

@MikeMcQuaid MikeMcQuaid merged commit aaf8c41 into Homebrew:master Sep 9, 2016
@MikeMcQuaid MikeMcQuaid deleted the diagnostic-duplicate-remove branch September 9, 2016 12:37
@BrewTestBot BrewTestBot removed the in progress Maintainers are working on this label Sep 9, 2016
@ilovezfs
Copy link
Copy Markdown
Contributor

ilovezfs commented Sep 9, 2016

@MikeMcQuaid thanks!

@Homebrew Homebrew locked and limited conversation to collaborators May 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants