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

Fixed from source install command for ign-fuel-tools7 #404

Merged
merged 1 commit into from
Mar 28, 2024

Conversation

peci1
Copy link
Contributor

@peci1 peci1 commented Mar 13, 2024

🦟 Bug fix

Summary

Newer gz-fuel-tools tutorials already contain a notice that a branch should be selected. The ign-fuel-tools7 tutorial is missing this.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@peci1 peci1 requested a review from nkoenig as a code owner March 13, 2024 12:50
@github-actions github-actions bot added the 🏯 fortress Ignition Fortress label Mar 13, 2024
@ahcorde ahcorde enabled auto-merge (squash) March 13, 2024 14:37
@mjcarroll mjcarroll disabled auto-merge March 28, 2024 15:10
@mjcarroll mjcarroll merged commit 2feefcb into gazebosim:ign-fuel-tools7 Mar 28, 2024
6 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏯 fortress Ignition Fortress
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants