-
Notifications
You must be signed in to change notification settings - Fork 1
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
🐍 Add CI-CD jobs for releasing to PyPI and Github #96
Merged
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
…different auth approaches add a username evaluation function (pn_user) and replace pn.state.user with pn_user(config), thus allowing username cleaning steps (e.g. email domain removal) move config related to basic authentication in a dedicated config file (e.g. config.server.auth_provider become config.basic_auth.auth_provider) basic auth attribute are available only if the server is set to basic_auth now the whole config passed as parameter dict to panel.serve is instantiated by hydra so that object inside the config are recursively instantiated add server/common.yaml has a _target_ attribute and a _conversion_ attribute add a check to test if basic authentication is active to set_guest_user_password (otherwise it may search for inexistent config attributes add checks on is_guest and is_admin functions to show the right sodget if authentication is not active (server=no_oauth) add a check on authorize callback to authorize every user if authentication is not active add config file for azure oauth add a new group of config for basic_auth (is an optional group, used only for basic_auth at the moment)
basic_auth config key has been moved at the root level of the main config
…t of is_basic_auth_active
group env variables in categories
merge pre-commit and commitizen environments under ci-cd environment improve makefile with meaningful names and additional commands adapt setup_dev_env.sh and compose_init.sh to the new ci-cd environment
…uilding process update pyproject.toml add an OmegaConf resolver to find data-lunch path and use it in config path (when required) change relevant filepaths to absolute paths pointing to the installed package (using the new 'pkg_path' OmegaConf resolver) change psycopg to binary (in requirements.txt) to avoid problems with pip install add a MANIFEST.in to include required resources in distribution files
…unch select a shorter and simpler name for PyPI data_lunch_app is converted to dlunch project name is set to dlunch PANEL_APP env variable and docker image name are left as data_lunch_app
Add PyPI package build for tagged commit on master Add TestPyPI package build for each push on development move docker build to ci_cd.yaml and rename release action to bump_version fix and improve makefile with commands to build an publish a PyPI package
remove CUSTOM_GITHUB_TOKEN where not required
external services can assess if the webapp is alive by calling '/health'
updates: - [github.com/pycqa/flake8: 6.1.0 → 7.0.0](PyCQA/flake8@6.1.0...7.0.0)
…fter version bump
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.
Feature
Fix
psw_special_chars
is_basic_auth_active
Refactor
Change package name from
data_lunch_app
todlunch
.Build
Docs
Add missing env variables to
README.md
.CI
manual_build.yaml
to test-releasemakefile
utility commands to install package from PyPI and TestPyPI