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

fix: update RULES.md with missing rule #1330

Merged
merged 6 commits into from
Feb 16, 2023

Conversation

briandonahue
Copy link
Collaborator

@briandonahue briandonahue commented Feb 10, 2023

Summary:

Closes #1329. Missing rule definition from PR #1321

Expected behavior:

Rule should have been added to RULES.md

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with gradle test to make sure you didn't break anything
  • Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
  • Linked all relevant issues
  • Include screenshot(s) showing how this pull request works and fixes the issue(s)

RULES.md Outdated Show resolved Hide resolved
RULES.md Outdated Show resolved Hide resolved
Copy link
Contributor

@isabelle-dr isabelle-dr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor changes in wording, LGTM!

RULES.md Outdated Show resolved Hide resolved
RULES.md Outdated Show resolved Hide resolved
isabelle-dr and others added 3 commits February 14, 2023 11:41
Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>
Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>
@bdferris-v2
Copy link
Collaborator

@fredericsimard I don't think it's enough for you to approve this PR. I believe you have to be the one to merge it as well.

@fredericsimard
Copy link
Contributor

@bdferris-v2 I didn't know what the last step was, hence why I didn't merge it:

  • Include screenshot(s) showing how this pull request works and fixes the issue(s)

@fredericsimard fredericsimard merged commit df8f221 into master Feb 16, 2023
@fredericsimard fredericsimard deleted the issue/1329/rulesmd_update branch February 16, 2023 15:18
KClough pushed a commit to JarvusInnovations/gtfs-validator that referenced this pull request Feb 24, 2023
* Rules.md udpate

* alignment tweaks

* Update RULES.md

Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>

* Update RULES.md

Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>

---------

Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>
KClough pushed a commit to JarvusInnovations/gtfs-validator that referenced this pull request Mar 1, 2023
* Rules.md udpate

* alignment tweaks

* Update RULES.md

Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>

* Update RULES.md

Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>

---------

Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>
ryon pushed a commit to JarvusInnovations/gtfs-validator that referenced this pull request Apr 1, 2023
* Rules.md udpate

* alignment tweaks

* Update RULES.md

Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>

* Update RULES.md

Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>

---------

Co-authored-by: isabelle-dr <isabelle@mobilitydata.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update RULES.md with after PR1321
4 participants