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.

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

Fix generator #105

Merged
merged 2 commits into from
Oct 14, 2022
Merged

Fix generator #105

merged 2 commits into from
Oct 14, 2022

Conversation

p-fruck
Copy link
Owner

@p-fruck p-fruck commented Oct 14, 2022

Pipeline was broken the last few days. The following actions have been taken to (hopefully) fix the pipeline:

  • pin openapi-generator-cli version to v6.1.0 for now as stable-release does not seem so stable...
  • add Makefile for ease of use
  • properly clean project directory before generating a new API client version
  • Install dependencies after API client generation as requirements.txt gets generated during build
  • Finally run unit tests

This PR has the following side effect:

  • Cleaned up old unused files
  • Adjusted naming conventions in a multitude of files

@p-fruck p-fruck self-assigned this Oct 14, 2022
@p-fruck p-fruck added bug Something isn't working automation labels Oct 14, 2022
@p-fruck p-fruck merged commit ac35843 into main Oct 14, 2022
@p-fruck p-fruck deleted the fix/generator branch October 14, 2022 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@p-fruck p-fruck

Labels
automation bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@p-fruck