Skip to content

Add section on downloading binaries #28

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

Merged
merged 4 commits into from
Dec 15, 2021

Conversation

milancurcic
Copy link
Member

I put this at the top of the page thinking that newcomers would probably want to try this first. But it can move elsewhere. It may need some help/addition from somebody who knows Windows to fact-check it and/or correct it.

Copy link
Member

@LKedward LKedward left a comment

Choose a reason for hiding this comment

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

Thanks @milancurcic, looks great!

@@ -1,7 +1,31 @@
# Installing fpm

This how-to guide covers the installation of the Fortran package manager (fpm) on various platforms.
This how-to guide covers the installation of the Fortran Package Manager (fpm) on various platforms.
Copy link
Member

Choose a reason for hiding this comment

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

I would keep this lowercase

Copy link
Member

Choose a reason for hiding this comment

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

This is actually a general question since it should be consistent for all documents. Maybe we can do this change separately rather than in three different patches? This way we can also adapt the translation files in the same patch.

Copy link
Member Author

Choose a reason for hiding this comment

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

We can do it in separate patches.

The choice boils down to whether we mean "Fortran Package Manager" as proper noun (I think so, and it allows abbreviating to fpm), in which case the correct spelling is to capitalize each word.

Copy link
Member

Choose a reason for hiding this comment

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

Let's make a separate patch and replace it in all files, maybe an automatic replacement in all files will work sufficiently well (also capturing the msgid's in the po files).

Copy link
Member

Choose a reason for hiding this comment

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

I agree with Milan, we're referring to fpm as a proper noun and hence it should be capitalized as such.

@milancurcic
Copy link
Member Author

Okay, capitalization should now be removed from all 3 PRs.

@awvwgk awvwgk merged commit 4ec7cc1 into fortran-lang:main Dec 15, 2021
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.

3 participants