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.

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

chore: update everything according to the latest standards #791

Merged
merged 9 commits into from
Jul 24, 2023
Merged

Conversation

artemrys
Copy link
Member

@artemrys artemrys commented Jul 24, 2023

Updates:

  • use meta job to determine Splunk versions to test against
  • use Poetry==1.4.2 in the CI
  • update release jobs to be consistent across the board
  • update some of the dependencies (pin Faker to be <19.0.0, they dropped support for Python 3.7)

For now we are ignoring an error specific to KVStore in SSG app.

@codecov-commenter
Copy link

codecov-commenter commented Jul 24, 2023

Codecov Report

Merging #791 (565e0d6) into main (56bb0ae) will not change coverage.
The diff coverage is n/a.

❗ Current head 565e0d6 differs from pull request most recent head 1c934de. Consider uploading reports for the commit 1c934de to get more accurate results

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main     #791   +/-   ##
=======================================
  Coverage   86.75%   86.75%           
=======================================
  Files          51       51           
  Lines        2763     2763           
=======================================
  Hits         2397     2397           
  Misses        366      366           

@artemrys artemrys marked this pull request as ready for review July 24, 2023 18:40
@artemrys artemrys merged commit c44ff2c into main Jul 24, 2023
32 of 33 checks passed
@artemrys artemrys deleted the update branch July 24, 2023 18:40
mbruzda-splunk pushed a commit that referenced this pull request Jul 25, 2023
* chore: update everything according to the latest standards

* chore: forgot about .addonmatrix

* ci: remove codecov integration

* chore: wait for Splunk's KVStore to be ready

* chore: attach credentials when making request to KVStore API

* style: pre-commit run --all-files

* chore: actually use response.json() instead of response

* chore: update list of internal errors to ignore error from splunk_secure_gateway app

* chore: remove KVStore additional verification
@srv-rr-github-token
Copy link
Contributor

🎉 This PR is included in version 5.0.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants