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/cli upd #186

Closed
wants to merge 42 commits into from
Closed

Fix/cli upd #186

wants to merge 42 commits into from

Conversation

vykozlov
Copy link
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also
include relevant motivation and context. List any dependencies that are
required for this change. Please note that there may be some sections that may
not apply to your change, in that case feel free to delete them.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • This change is a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide
instructions so we can reproduce. Please also list any relevant details for
your test configuration

  • Test A
  • Test B

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

IgnacioHeredia and others added 30 commits June 7, 2024 11:13
@IgnacioHeredia @alvarolopez
fix: make description optional
77d3985
@IgnacioHeredia @alvarolopez
feat: allow multiple input files
47440d9
@IgnacioHeredia @alvarolopez
refactor: remove unneeded check
28f2127 
We are already mapping these fields
@IgnacioHeredia @alvarolopez
feat: allow linebreaks in help messages
4fa01aa
@IgnacioHeredia @alvarolopez
feat: add required info to help message
e610071
@IgnacioHeredia @alvarolopez
feat: add types to help message
c4b9a57
@IgnacioHeredia @alvarolopez
feat: properly support lists and dicts in argparse
f52101d
@IgnacioHeredia @alvarolopez
style: organize _fields_to_dict code
216f71a
@IgnacioHeredia @alvarolopez
feat: support bools
cb1d579
@IgnacioHeredia @alvarolopez
style: fix flake8
13e6105
@IgnacioHeredia @alvarolopez
style: fix black
53a4c32
@alvarolopez
Merge branch 'master' into fix/cli
60257d1
@vykozlov
fix: studying apispec for fields conversion
c51c3b4
@vykozlov
fix: store BytesIO object in file
12a1d46
@alvarolopez
ci: watch also 2.x branch for releases
12ec967
@alvarolopez @IgnacioHeredia
tests: fix missing setuptools dependency
627b89a
@alvarolopez @IgnacioHeredia
feat: add -dev suffix to the version when installed as dev
4d087b3
@alvarolopez @IgnacioHeredia
fix: remove deprecated cli option
05d7f06
@IgnacioHeredia
fix: make description optional
7717348
@IgnacioHeredia
feat: allow multiple input files
b7b1b82
@IgnacioHeredia
refactor: remove unneeded check
9eab0cc 
We are already mapping these fields
@IgnacioHeredia
feat: allow linebreaks in help messages
2deabdb
@IgnacioHeredia
feat: add required info to help message
d710254
@IgnacioHeredia
feat: add types to help message
6099f24
@IgnacioHeredia
feat: properly support lists and dicts in argparse
071b777
@IgnacioHeredia
style: organize _fields_to_dict code
1bd60f4
@IgnacioHeredia
feat: support bools
6e1453d
@IgnacioHeredia
style: fix flake8
ac8f874
@IgnacioHeredia
style: fix black
be440bf
@alvarolopez
fix: change target branch on release-please action
32b669c
alvarolopez and others added 12 commits August 6, 2024 12:53
@alvarolopez
fix: change deprecated GH action
71e8230
@alvarolopez
fix: allow users to setup base_path for serving under a different prefix
39fa910 
Fixes #111
@alvarolopez
ci: fix glob pattern
6c1a311
@ai4-bot @alvarolopez
chore(releases/2.x): release 2.5.0
aa6cba7
@alvarolopez
ci: fix package build, using trusted publisher
07d7371
@alvarolopez
Revert "ci: fix package build, using trusted publisher"
b142897 
This reverts commit f619ebd.
@alvarolopez
ci: fix CI build and publish
1099b6a
@alvarolopez @IgnacioHeredia
fix: do not pass basePath to aiohttp_apispec
45c8452 
Passing the basePath to the aiohttp_apispec.setup_aiohttp_apispec method
caused that URL that were used from the Swagger UI added the path twice,
resulting in 404 not found methods.

Fixes #111
@alvarolopez @IgnacioHeredia
fix: also serve static_path from base_path
267ab6a 
Fixes #111
@ai4-bot @alvarolopez
chore(releases/2.x): release 2.5.1
2955a63
@vykozlov
fix: prepare deepaas-cli for the release 2.x
1d9facd
@vykozlov
fix: merge from releases/2.x, replace return with log.info()
35033e4
@vykozlov
Copy link
Contributor Author

wrongly made it for "master", closing PR

@vykozlov vykozlov closed this Aug 26, 2024
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.
4 participants
@vykozlov @IgnacioHeredia @alvarolopez @ai4-bot