[go: up one dir, main page]

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

CI: track binary size changes over time #2634

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

pjonsson
Copy link
Contributor

Add a step in the CI that shows the size
difference for each PR.

This requires keeping track of previous
sizes, so also add a step that pushes
sizes.log to a data repository when
merging PRs to develop.

There is a small window where two jobs
can push at the same time. Instead of adding
some complicated locking, just let one of
the jobs fail with the push, but make the job
green.

This requires a new repository to be created,
and an access token with write permissions
for that repository. The access token also needs
to be added as a secret to this CI with the name
DATA_PUSH_TOKEN.

Here's a screenshot of what it looks like when saving the output from "date":

image

Add a step in the CI that shows the size
difference for each PR.

This requires keeping track of previous
sizes, so also add a step that pushes
sizes.log to a data repository when
merging PRs to develop.
@pjonsson
Copy link
Contributor Author
pjonsson commented Sep 1, 2023

Use the contiki-ng.github.io repository to store data.

This PR needs a key with read/write permissions on repository on contiki-ng.github.io, and the key put in a secret called DATA_PUSH_TOKEN here.

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.

1 participant