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

feat(build): Add Git archives version files #209

Closed
wants to merge 2 commits into from

Conversation

njzjz-bot
Copy link

When downloading the source code of a branch or a release from GitHub or the mirror, there is no Git version information, making setuptools-scm unable to read the version. The alternative way is to add a git archive file supported by setuptools-scm. See:

Generated by the task: njzjz-bot/njzjz-bot#4.

anyangml and others added 2 commits April 11, 2024 01:17
`var_name` is no longer required as user input for `polar` and `dipole`.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
When downloading the source code of a branch or a release from GitHub or the mirror, there is no Git version information, making setuptools-scm unable to read the version. The alternative way is to add a git archive file supported by setuptools-scm. See:
* https://setuptools-scm.readthedocs.io/en/latest/usage/#git-archives (available in setuptools-scm 7.0.0)
* https://git-scm.com/docs/git-archive#Documentation/git-archive.txt-export-subst
* https://docs.github.com/en/repositories/working-with-files/using-files/downloading-source-code-archives

Generated by the task: njzjz-bot/njzjz-bot#4.
@pep8speaks
Copy link

Hello @njzjz-bot! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 222:80: E501 line too long (95 > 79 characters)
Line 229:80: E501 line too long (88 > 79 characters)
Line 304:80: E501 line too long (81 > 79 characters)
Line 380:80: E501 line too long (87 > 79 characters)
Line 400:80: E501 line too long (80 > 79 characters)
Line 401:80: E501 line too long (91 > 79 characters)
Line 459:80: E501 line too long (88 > 79 characters)
Line 497:80: E501 line too long (86 > 79 characters)
Line 501:80: E501 line too long (95 > 79 characters)
Line 512:80: E501 line too long (91 > 79 characters)
Line 517:80: E501 line too long (80 > 79 characters)
Line 525:80: E501 line too long (80 > 79 characters)
Line 536:80: E501 line too long (87 > 79 characters)
Line 544:80: E501 line too long (88 > 79 characters)
Line 559:80: E501 line too long (81 > 79 characters)
Line 561:80: E501 line too long (88 > 79 characters)
Line 562:80: E501 line too long (87 > 79 characters)

Line 129:80: E501 line too long (81 > 79 characters)
Line 182:80: E501 line too long (86 > 79 characters)
Line 229:80: E501 line too long (85 > 79 characters)
Line 237:80: E501 line too long (80 > 79 characters)
Line 256:80: E501 line too long (82 > 79 characters)
Line 282:80: E501 line too long (81 > 79 characters)
Line 335:80: E501 line too long (86 > 79 characters)
Line 382:80: E501 line too long (85 > 79 characters)
Line 390:80: E501 line too long (80 > 79 characters)
Line 404:80: E501 line too long (81 > 79 characters)
Line 406:80: E501 line too long (82 > 79 characters)

@njzjz njzjz closed this Apr 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants