update from main #166
Merged
update from main #166
Conversation
This file contains 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
* Basic logic for token checking * Initial Firebase Setup * Implemented token system * Linting checks * Linting checks * firebase_admin dependency added * added requirement * firebase_admin added to specific requirements * adds service account certificate env var from github secrests * removes superflous dependency 'pnmlbpmntransformer' * 💚 * 💚 Changes service account certificate env var handling * Check Tokens Function added * adds basic decodation of base64 service account certificate * Refactor service account certificate handling * fixes set_force_std_xml flag position * now uses correct var for decoding base64 * added tempfile for secret * remove unused dependency * test prints added * test * test * test * test * test * test * test * test * test * test * test * test * t * test * test * test * test * test * improved code quality * final touches * fixes cd pipeline: sets necessary env var for gcp service account certificate * fix cd pipeline: also sets env var in matrix job * replaces print message for missing env var with exception --------- Co-authored-by: Jeldrik Merkelbach <[email protected]> Co-authored-by: Jeldrik Merkelbach <[email protected]>
* Basic logic for token checking * Initial Firebase Setup * Implemented token system * Linting checks * Linting checks * firebase_admin dependency added * added requirement * firebase_admin added to specific requirements * adds service account certificate env var from github secrests * removes superflous dependency 'pnmlbpmntransformer' * 💚 * 💚 Changes service account certificate env var handling * Check Tokens Function added * adds basic decodation of base64 service account certificate * Refactor service account certificate handling * fixes set_force_std_xml flag position * now uses correct var for decoding base64 * added tempfile for secret * remove unused dependency * test prints added * test * test * test * test * test * test * test * test * test * test * test * test * t * test * test * test * test * test * improved code quality * final touches * fixes cd pipeline: sets necessary env var for gcp service account certificate * fix cd pipeline: also sets env var in matrix job * replaces print message for missing env var with exception * swaps checkout and env set step in cd pipeline * removes workflow dispatch and finally fixes CD env var (hopefully) --------- Co-authored-by: Jeldrik Merkelbach <[email protected]> Co-authored-by: Jeldrik Merkelbach <[email protected]>
nilsbgt
added a commit
that referenced
this pull request
Jul 8, 2024
* update from main (#166) * get main (#168) * e2e test transformer * e2e test transformer * e2e test transformer * e2e test transformer * e2e test transformer * e2e test transformer * e2e test transformer * e2e tests * e2e tests# * e2e tests# * e2e tests# * changed data for bpmntopnml test * changed filepath * get main (#189) * test * test1 * test2 * added logger * added logger * try stdout * try print * wip: adds xml normalization for assertions * fixes e2e test assertion of xml * fix ruff errors * fix file input * remove integrationtest boilerplate code and refresh Tokens function * changed expected xml * xml beautiful * pnml_to_bpmn should work now * other direction test * fix bpmntopnml * sort xmls * fix file input --------- Co-authored-by: nils.baumgartner <[email protected]> Co-authored-by: Niyada <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.