Skip to content

Apply latest merges for our convenience #416

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.

Sign up for GitHub

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

Jump to bottom
Closed
led02 wants to merge 24 commits into from
Closed

Apply latest merges for our convenience #416

led02 wants to merge 24 commits into from

Conversation

@led02
Copy link
Member

@led02 led02 commented Aug 1, 2025

#347 changes pyproject.toml and this branch needs to adapt it, too.

sdruskat and others added 20 commits July 31, 2025 23:05
@sdruskat
Test user agent name
08e1c23 
- Make sure user agent name in utils is a combination of package name,
version and homepage URL:

#347 (comment)
@sdruskat
Use distribution version in test
8bf6343
@sdruskat
Update pyproject.toml to adhere to PEP 621
9ff39d7
@sdruskat
Fix retrieving URLs from distribution metadata
d83c93c
@sdruskat
Remove superfluous poetry-related sections
9dfcd30
@sdruskat
Fix Poetry build backend version
45435dc
@sdruskat
Make dev dependencies fully optional
5ac060e
@sdruskat
Restore poetry-defined dependency groups
ca4a11c
@sdruskat
Change authorship back to align with CFF
f90cb4d
@sdruskat
Reset authorship policy to align with CFF
b46a7a5
@sdruskat
Update lock
a2e20e0
@sdruskat
Disambiguate var name
6a95ebf
@sdruskat
Update poetry version to use for docs build
c90002a
@sdruskat
Update readthedocs config
eb1dfca 
- Build failed
- RTD docs provide guidance for building with poetry:
https://test-builds.readthedocs.io/en/poetry/
- This change uses the guidance
@sdruskat
Replace link to CFF file with absolute URL to fix RTD build
e26fb3b
@sdruskat
Reduce scope of import
588f372
@sdruskat @led02
Use concrete path of test file to read file into toml
f149a9a 
Co-authored-by: Michael Meinel <michael.meinel@dlr.de>
@sdruskat
Import Path from pathlib
d04a47e
@sdruskat
Format comprehension for better readability
4fd43a9
@sdruskat
Merge pull request #407 from softwarepub/feature/347-update-build_bac…
b0b84a7 
…kend

Fix #347: Rebuild `pyproject.toml` to adhere to PEP 621 & update poetry version
@led02
Copy link
Member Author

led02 commented Aug 1, 2025 •
edited
Loading

Motivational post... after applying

there is promising output from pytest:

src\hermes\model\__init__.py                                0      0   100%
src\hermes\model\context_manager.py                        48      6    88%
src\hermes\model\errors.py                                 10     10     0%
src\hermes\model\types\__init__.py                         11      0   100%
src\hermes\model\types\ld_container.py                    105      9    91%
src\hermes\model\types\ld_context.py                       33      3    91%
src\hermes\model\types\ld_dict.py                          77     54    30%
src\hermes\model\types\ld_list.py                          47     30    36%
src\hermes\model\types\pyld_util.py                        68      5    93%
src\hermes\utils.py                                        12      0   100%

@sdruskat
Copy link
Contributor

sdruskat commented Aug 5, 2025

Sorry, broke the PR 🙈. Fixing conflicts in pyproject.toml merged the whole bit back to develop 🤷. Reverting made a mess. Will re-open after cherry-picking 😅.

@sdruskat sdruskat closed this Aug 5, 2025
@sdruskat sdruskat mentioned this pull request Aug 5, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@led02 @sdruskat