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

Switch to GitHub actions and cleanup syntax/API use #48

Merged
merged 2 commits into from
Jul 12, 2024
Merged

Conversation

mikix
Copy link
Contributor

@mikix mikix commented Jul 12, 2024

  • Switch from external Travis to GitHub actions for CI
  • Update a jinja2 import to avoid removed methods (removal in 3.1, new method appeared in 3.0 - so bump requirement to 3.0 too)
  • Fix a few python syntax warnings.
  • Update README's mention of what this repo can parse.
  • Fix a Practitioner typo in a comment. Fix by Kevin Wood (kwood16).

Fixes #45
Fixes #47

- Update a jinja2 import to avoid removed methods (removal in 3.1,
  new method appeared in 3.0 - so bump requirement to 3.0 too)
- Fix a few python syntax warnings.
- Update README's mention of what this repo can parse.
- Fix a Practitioner typo in a comment. Fix by Kevin Wood (kwood16).
@mikix mikix force-pushed the mikix/modernization branch from 92b2544 to 702b066 Compare July 12, 2024 12:45
README.md Outdated
The `master` branch is currently capable of parsing _STU 3, v3.0.0_.
The `develop` branch should be capable of parsing the continuous integration build and will be merged into master on new major FHIR releases.
There may be tags for specific freezes, see [releases](https://github.com/smart-on-fhir/fhir-parser/releases).
The `master` branch is currently capable of parsing _R5_.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The develop branch is strictly 14 commits behind master currently - folks seem to have switched to trunk-based dev (which seems fine, since it's currently backwards compatible).

I took out the link to releases because (a) there are no release, just tags and (b) even the tags aren't very up to date - just a bunch of 1.x stuff from early days.

@mikix mikix marked this pull request as ready for review July 12, 2024 12:47
@mikix mikix changed the title Fix various syntax issues and a jinja2 API break Switch to GitHub actions and cleanup syntax/API use Jul 12, 2024
Comment on lines -1 to -4
Jinja2>=2.9.5
MarkupSafe==0.23
colorlog>=2.10.0
jinja2>=3.0
requests>=2.13.0
colorlog==2.10.0
Copy link
Contributor Author

Choose a reason for hiding this comment

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

MarkupSafe didn't seem to be used anymore, so I removed it (and loosened the pins on the other requirements)

@mikix mikix force-pushed the mikix/modernization branch from 3fb65f7 to 94adfa1 Compare July 12, 2024 13:34
@mikix mikix merged commit 5c608f5 into main Jul 12, 2024
5 checks passed
@mikix mikix deleted the mikix/modernization branch July 12, 2024 13:36
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.

Can't locate Sample files Enable Travis repo
2 participants