-
Notifications
You must be signed in to change notification settings - Fork 247
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
Adding Build script for chalice on UBI #1019
Adding Build script for chalice on UBI #1019
Conversation
Test 1:- sh chalice_ubi_8.3.sh 1.22.4 Cloning into 'chalice'... You are in 'detached HEAD' state. You can look around, make experimental If you want to create a new branch to retain commits you create, you may git switch -c Or undo this operation with: git switch - Turn off this advice by setting config variable advice.detachedHead to false HEAD is now at 1ee44df Bumping version to 1.22.4 FLAKE8No unused imports, no undefined vars,flake8 --ignore=E731,W503,W504 --exclude chalice/init.py,chalice/compat.py,chalice/vendored/botocore/regions.py --max-complexity 10 chalice/ Proper docstring conventions according to pep257pydocstyle --add-ignore=D100,D101,D102,D103,D104,D105,D204,D301 --match='(?!(test_|regions)).*.py' chalice/ PYLINTpylint --rcfile .pylintrc chalice --load-plugins tests.codelinter Your code has been rated at 10.00/10 Run our custom linter on test code.pylint --load-plugins tests.testlinter --disable=I,E,W,R,C,F --enable C9999,C9998 tests/ Your code has been rated at 10.00/10 py.test --cov chalice --cov-report term-missing tests/unit tests/functional tests/integration tests/unit/test_analyzer.py ................................................................. [ 4%] =========================================================================== warnings summary =========================================================================== tests/unit/test_package.py::test_can_create_app_packager -- Docs: https://docs.pytest.org/en/latest/warnings.html ----------- coverage: platform linux, python 3.6.8-final-0 -----------
|
Test 2: sh chalice_ubi_8.3.sh FLAKE8No unused imports, no undefined vars,flake8 --ignore=E731,W503,W504 --exclude chalice/init.py,chalice/compat.py,chalice/vendored/botocore/regions.py --max-complexity 10 chalice/ Proper docstring conventions according to pep257pydocstyle --add-ignore=D100,D101,D102,D103,D104,D105,D107,D204,D301 --match='(?!(test_|regions)).*.py' chalice/ PYLINTpylint --rcfile .pylintrc chalice --load-plugins tests.codelinter Your code has been rated at 10.00/10 Run our custom linter on test code.pylint --load-plugins tests.testlinter --disable=I,E,W,R,C,F --enable C9999,C9998 tests/ Your code has been rated at 10.00/10 py.test --cov chalice --cov-report term-missing tests/unit tests/functional tests/integration tests/unit/test_analyzer.py ................................................................. [ 4%] =========================================================================== warnings summary =========================================================================== tests/unit/test_package.py::test_can_create_app_packager tests/functional/test_local.py::test_can_import_env_vars -- Docs: https://docs.pytest.org/en/stable/warnings.html ----------- coverage: platform linux, python 3.6.8-final-0 -----------
|
c/chalice/chalice_ubi_8.3.sh
Outdated
# Dependency installation | ||
sudo dnf install python36 -y | ||
sudo dnf install -y git make python3-devel openssl-devel.ppc64le gcc | ||
dnf install nodejs npm -y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't we need to use "sudo" for running "dnf install" ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeap ,sudo is required and added.
fae064e
to
17493e0
Compare
Yes , This has been tested on master and 1.22.4 . |
Can we find out what is the master version and list it in the header |
17493e0
to
4ab1d12
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we find out what is the master version and list it in the header
done..added 1.21.8 also as there is another required version for this package in row 154 on spreadsheet.
c/chalice/chalice_ubi_8.3.sh
Outdated
# Dependency installation | ||
sudo dnf install python36 -y | ||
sudo dnf install -y git make python3-devel openssl-devel.ppc64le gcc | ||
dnf install nodejs npm -y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeap ,sudo is required and added.
.added master and 1.21.8 also as there is another required version for this package in row 154 on spreadsheet. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
No description provided.