Skip to content

Updating common.rst#15

Merged
iparask merged 3 commits into
masterfrom
gp/update_common
Jul 7, 2026
Merged

Updating common.rst#15
iparask merged 3 commits into
masterfrom
gp/update_common

Conversation

@iparask

@iparask iparask commented Jun 25, 2026

Copy link
Copy Markdown
Member

I updated common section to be a bit more general. Parts that where python specific will move there for correctness and completeness.

I also updated how pages are linked and referenced to be in a more RST style.

Depends on #14 to fix the errors.

Updated common.rst with newest instructions about git, versioning and
other small information. Also, I'm moving versioneer under python,
as it not general enough.
@iparask iparask requested review from JBorrow and mhasself June 25, 2026 19:45

@mhasself mhasself left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Minor comments inline.

Comment thread source/common.rst Outdated
Comment on lines +77 to +79
Furthermore, we will provide configuration files for most linters and
formatter, so that you can easily setup your editor and to automatically
format your code to the style guidelines.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Such promises for the future are not really helpful. It might be good here to say "find config files on this wiki page ", and start that wiki page, and put things on there (whatever you have so far).

Comment thread source/common.rst Outdated
Do not add temporary files, data files, backup files, or compilation products
to a repository! When preparing to commit code, run ``git status`` frequently
to see what files are present, modified, and staged for commit. Commands like
``git add *`` and ``git commit -a`` should not be used and are higly

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

highly

Comment thread source/common.rst Outdated
<https://git-scm.com/docs/gitignore>`__. Note that github makes
`templates <https://github.com/github/gitignore>`__ available for many
different programmaing languages.
documentation on the `.gitingore file`_. Note that github makes

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

gitignore, not gitingore! my bad :)

Comment thread source/common.rst Outdated
`templates <https://github.com/github/gitignore>`__ available for many
different programmaing languages.
documentation on the `.gitingore file`_. Note that github makes
`templates`_ available for many different programmaing languages.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

programming

Comment thread source/common.rst
new features or fix a significant number of issues to the code, without
changing the API. In the case of releases that are primarily bug fixes or
rolling releases, the patch version z should be incremented.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe at least mention some tools that are used to record and expose version numbers? In SO we lean into the github release feature, and versioneer, right?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Okay, I will add that we lean in the Github release feature, but I am moving versioneer under python. I have added a similar tool under the C++ part of the docs if you see #14 again.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For what it's worth I've moved away from versioneer in the packages I manage. It only supports setuptools as a build backend -- and a while back setuptools made some breaking changes that annoyed me enough to switch to something more modern (I chose hatchling). I'm using hatch-vcs now in place of versioneer.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Personally I would stay away from auto-versioning tools... "Not having to think about what version this is" causes more problems than it's worth.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I also use setuptools-scm instead of versioneer.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great, in the appropriate sections just suggest one or more good options (including something simple). The point is to point people in the right direction and discourage wheel-reinvention / rabbit-hole-excursion.

@iparask iparask mentioned this pull request Jun 29, 2026
@iparask iparask merged commit 4b674ae into master Jul 7, 2026
1 check passed
@iparask iparask deleted the gp/update_common branch July 7, 2026 13:48
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.

4 participants