Repository navigation
build: use Makefile for generating, building, checking documentation, and running formatting #78
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
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
448f2b1
Use Makefile for generating, building, checking documentation, and ru…
stevepiercy fa48bb6
Fixed .gitignore to exclude docs build dir
stevepiercy 4479cce
REUSE
stevepiercy 09ced34
Fix RTD build
stevepiercy fd16a43
Remove `-W` option so that warnings aren't treated as errors.
stevepiercy 13d7634
Remove unchecked for building docs
stevepiercy f5edaa3
Move documentation into its own CI
stevepiercy a6ef015
REUSE
stevepiercy a93fc6a
Enable changes-check target for news fragments
stevepiercy 4e1b8c8
Enable changes-check target for news fragments
stevepiercy 2f4a294
Fix zizmor check
stevepiercy f12b248
news
stevepiercy 9d5f599
Clean up `pyproject.toml` and add `tool.uv` configuration.
stevepiercy 0e6864f
docs: clarify delete_event's old_state/new_state sentence
SashankBhamidi 52a009f
Remove redundant news fragments check and rename
stevepiercy 35a8246
Rewire towncrier check to changes-check target
stevepiercy 4282185
Install uv for changes-check target
stevepiercy b94d4f4
quote opts
stevepiercy f213669
Fix comment paths
stevepiercy 547bc10
Switch from pip to uv to lint and format code
stevepiercy 2db8d98
Switch from pip to uv to run type checking and add new target to Make…
stevepiercy a6f017a
Switch from pip to uv to run tests with coverage
stevepiercy dfe4526
Add prerequisites to Contribute documentation.
stevepiercy 572618a
- Add post-installation and uv to accept.txt
stevepiercy 526ddb9
fix: restore Vale.Terms and uncomment project vocab, fixing CI
SashankBhamidi 451102e
fix: dead requests extra, unmigrated integration CI, dropped -W, leak…
SashankBhamidi 7cd24dc
fix: broken make targets, stale paths, wrong Sphinx role, and track u…
SashankBhamidi 3c6312b
fix: anchor the private-base regex and guard against a non-string base
SashankBhamidi 589eb98
docs: split the news/63 fragment per behavior and add the .rst extension
SashankBhamidi ba4ed98
Apply suggestion from @SashankBhamidi
SashankBhamidi a24aa28
docs: apply reviewer's news fragment prose suggestions
SashankBhamidi 132a481
Merge branch 'makefile-docs-uv' of https://github.com/pycalendar/cale…
SashankBhamidi 82ece5a
fix: revert pycalendar Open Collective link, it has no fiscal host yet
SashankBhamidi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,53 @@ | ||
| # SPDX-FileCopyrightText: 2026 calendaring-jmap contributors | ||
| # SPDX-License-Identifier: AGPL-3.0-or-later | ||
|
|
||
| name: Documentation build, check links, spelling, grammar, and style | ||
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| paths: | ||
| - 'docs/**' | ||
| # Build pull requests | ||
| pull_request: | ||
| paths: | ||
| - 'docs/**' | ||
| - 'Makefile' | ||
| - 'src/calendaring_jmap/*.py' | ||
| - 'src/calendaring_jmap/convert/*.py' | ||
| - 'src/calendaring_jmap/objects/*.py' | ||
| - '.github/workflows/documentation.yml' | ||
| - '.readthedocs.yaml' | ||
| - '.vale.ini' | ||
| - 'news/*.*' | ||
|
|
||
| jobs: | ||
| docs: | ||
| name: Documentation | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| python-version: ['3.11', '3.14'] | ||
| steps: | ||
| - uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1 | ||
| with: | ||
| fetch-depth: 0 | ||
| fetch-tags: true | ||
| persist-credentials: false | ||
|
|
||
| - name: Install uv | ||
| uses: astral-sh/setup-uv@4c58170a945e4e68ce3fc5b7af3d341d9b8c64ef # v10.3.3 | ||
| with: | ||
| version: "0.13.6" | ||
|
|
||
| - name: Create Python virtual environment, install requirements | ||
| run: make .venv PYTHONVERSION=${{ matrix.python-version }} | ||
|
|
||
| - name: Build HTML documentation | ||
| run: make html PYTHONVERSION=${{ matrix.python-version }} | ||
|
|
||
| - name: Run vale | ||
| run: make vale VALEOPTS="--minAlertLevel='warning'" PYTHONVERSION=${{ matrix.python-version }} | ||
|
|
||
| - name: Check for broken links | ||
| run: make linkcheckbroken PYTHONVERSION=${{ matrix.python-version }} | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,11 +1,20 @@ | ||
| # SPDX-FileCopyrightText: 2026 calendaring-jmap contributors | ||
| # SPDX-License-Identifier: AGPL-3.0-or-later | ||
|
|
||
| StylesPath = docs/.vale/styles | ||
| Vocab = Base | ||
| StylesPath = docs/styles | ||
| MinAlertLevel = suggestion | ||
| Packages = Microsoft, https://github.com/ammil-industries/vale-signs-of-ai-writing/releases/download/v0.1.1/signs-of-ai-writing.zip | ||
| Vocab = Base, calendaring-jmap | ||
| Packages = Microsoft | ||
|
|
||
| [*.rst] | ||
| BasedOnStyles = Vale, Microsoft, signs-of-ai-writing, calendaring-jmap | ||
| Vale.Terms = NO | ||
| BasedOnStyles = Vale, Microsoft, calendaring-jmap | ||
| Microsoft.Contractions = suggestion | ||
| Microsoft.GeneralURL = NO | ||
| # Override Microsoft.Quotes to allow a single letter in quotes followed by punctuation. | ||
| # docs/styles/icalendar/Quotes.yml | ||
| Microsoft.Quotes = NO | ||
| Microsoft.Units = suggestion | ||
|
|
||
| # Ignore Sphinx Python object roles, such as ":class:" and ":meth:", | ||
| # to prevent false positives from Microsoft.Spacing on dotted Python names. | ||
| TokenIgnores = :[a-z]+:\x60.+\x60 |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.