Skip to content

Mpy size actions #197

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Mpy size actions #197

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
c2d2270
adding mpy size actions comment
FoamyGuy Aug 18, 2022
4a8cbe2
add raw tags
FoamyGuy Aug 18, 2022
96fd6cb
update actions task for conditional comment
FoamyGuy Aug 25, 2022
f539521
fix raw tag
FoamyGuy Aug 25, 2022
730d4b0
use checkout action to clone
FoamyGuy Aug 25, 2022
5c3eb22
remove extra space
FoamyGuy Aug 29, 2022
dde0226
clone and build main branch for size measurements
FoamyGuy Sep 24, 2022
549f559
fix template var
FoamyGuy Sep 24, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 29 additions & 0 deletions {{ cookiecutter.__dirname }}/.github/workflows/build.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,35 @@ jobs:
pre-commit run --all-files
- name: Build assets
run: circuitpython-build-bundles --filename_prefix {% raw %}${{ steps.repo-name.outputs.repo-name }}{% endraw -%} {% if cookiecutter.target_bundle == 'Community' %} --package_folder_prefix {% if cookiecutter.library_prefix -%} {{ cookiecutter.library_prefix | lower | replace(' ', '_') }}_ {%- else %}""{%- endif -%}{%- endif %} --library_location .
- name: Clone Size Tools
uses: actions/checkout@v3
with:
repository: 'circuitpython/CircuitPython_Org_size_tools'
path: 'CircuitPython_Org_size_tools'
- name: Clone Main Branch
uses: actions/checkout@v3
with:
path: 'main_branch_repo'
ref: 'main'
- name: Build main
run: |
cd main_branch_repo
circuitpython-build-bundles --filename_prefix {% raw %}${{ steps.repo-name.outputs.repo-name }}{% endraw -%} --library_location .
- name: Measure Sizes
run: |
python CircuitPython_Org_size_tools/size_tools.py
- name: Check sizes.txt file existence
id: check_sizes_file
uses: andstor/file-existence-action@v1
with:
files: "sizes.txt"
- name: Size Measurement Comment
if: github.event_name == 'pull_request' && steps.check_sizes_file.outputs.files_exists == 'true'
uses: machine-learning-apps/pr-comment@master
env:
GITHUB_TOKEN: {% raw %}${{ secrets.GITHUB_TOKEN }}{% endraw %}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Latest commit removes the extra space here. This was copied from another similar usage of the token which has now been removed by #202

with:
path: sizes.txt
- name: Archive bundles
uses: actions/upload-artifact@v2
with:
Expand Down