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

GitHub script distribution #806

Merged
merged 3 commits into from
Jul 3, 2023
Merged

GitHub script distribution #806

merged 3 commits into from
Jul 3, 2023

Conversation

bjoernricks
Copy link
Contributor

What

Allow to run GitHub scripts as a module and ship our GitHub scripts with the pontos package.

Why

Distribute and install the GitHub scripts via the Python pontos package. This allows running the scripts without having to download them from the repository. This will simplify our actions.

Checklist

  • Tests

This will allow to included some GitHub script as a module and install
it into the current Python environment.
Allow to run the GitHub scripts directly from the pontos Python package.
@bjoernricks bjoernricks requested a review from a team as a code owner July 3, 2023 08:27
@bjoernricks bjoernricks enabled auto-merge (rebase) July 3, 2023 08:27
@github-actions
Copy link

github-actions bot commented Jul 3, 2023

Conventional Commits Report

Type Number
Changed 2

🚀 Conventional commits found.

@codecov
Copy link

codecov bot commented Jul 3, 2023

Codecov Report

Merging #806 (7bcacbb) into main (1845f1a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 7bcacbb differs from pull request most recent head b6487f5. Consider uploading reports for the commit b6487f5 to get more accurate results

@@           Coverage Diff           @@
##             main     #806   +/-   ##
=======================================
  Coverage   89.74%   89.75%           
=======================================
  Files          82       82           
  Lines        5421     5425    +4     
  Branches      762      764    +2     
=======================================
+ Hits         4865     4869    +4     
  Misses        389      389           
  Partials      167      167           
Impacted Files Coverage Δ
pontos/github/script/load.py 97.95% <100.00%> (+0.18%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

y0urself
y0urself previously approved these changes Jul 3, 2023
The scripts got moved to a pontos module.
@bjoernricks bjoernricks requested a review from y0urself July 3, 2023 08:35
@bjoernricks bjoernricks merged commit 63c5716 into main Jul 3, 2023
11 checks passed
@bjoernricks bjoernricks deleted the github-script-distribution branch July 3, 2023 08:36
wiegandm added a commit that referenced this pull request Aug 18, 2023
The location of the GitHub scripts changed in #806, this fix restores
functionality.
bjoernricks pushed a commit that referenced this pull request Aug 18, 2023
The location of the GitHub scripts changed in #806, this fix restores
functionality.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants