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

Update Building in Windows 10 or 11 with MSYS2.md #8248

Merged
merged 5 commits into from Jul 24, 2022

Conversation

pwnept
Copy link
Contributor

@pwnept pwnept commented Jul 19, 2022

Changed default git command to checkout with a tag instead of a branch, kept the old command modified as a comment. (because i.e. release_5.0.0 branch does not exist)
Added where to find tags.
Updated version numbers of iNav and ARM toolchain.

Changed default git command to checkout with a tag instead of a branch, but kept the old command as a comment. (i.e. release_5.0.0 branch does not exist)
Added where to find tags (Just in case).
Updated version numbers of iNav and ARM toolchain.
@erstec erstec added this to the 6.0 milestone Jul 19, 2022
Copy link
Contributor Author

@pwnept pwnept left a comment

Choose a reason for hiding this comment

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

Fixed in b48adf5

@MrD-RC
Copy link
Collaborator

MrD-RC commented Jul 24, 2022

INAV should be iNav. Look at the name of the repository iNavFlight.

@DzikuVx
Copy link
Member

DzikuVx commented Jul 24, 2022

@MrD-RC some time we decided it's INAV not to have any connotation with Apple. Organization is iNavFlight but INAV is INAV

@DzikuVx DzikuVx merged commit fed23cf into iNavFlight:master Jul 24, 2022
@MrD-RC
Copy link
Collaborator

MrD-RC commented Jul 24, 2022

@DzikuVx lower case i is not just Apple. Plenty of other companies and products also use the format. jQuery for example.

I'm not a fan of the uppercase, shouting version. The original was iNavFlight inav.

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.

None yet

4 participants