Skip to content

Sync website validation assets with template #108

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

Merged
merged 1 commit into from
Aug 16, 2021
Merged

Sync website validation assets with template #108

merged 1 commit into from
Aug 16, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Aug 15, 2021

We have assembled a collection of reusable project assets:
https://github.com/arduino/tooling-project-assets
These assets will be used in the repositories of all Arduino tooling projects.

Some minor improvements and standardizations have been made in the upstream "template" assets for checking the website
publishing system, and those are introduced to this repository here.

Notable:

  • Add paths filter to prevent unnecessary workflow runs
  • Bump Python version used by the workflow from 3.8 to 3.9

Some task names have changed:

  • docs:build -> website:check
  • docs:serve -> website:serve

@per1234 per1234 added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels Aug 15, 2021
@per1234 per1234 requested a review from umbynos August 15, 2021 08:55
We have assembled a collection of reusable project assets:
https://github.com/arduino/tooling-project-assets
These assets will be used in the repositories of all Arduino tooling projects.

Some minor improvements and standardizations have been made in the upstream "template" assets for checking the website
publishing system, and those are introduced to this repository here.

Notable:

- Add paths filter to prevent unnecessary workflow runs
- Bump Python version used by the workflow from 3.8 to 3.9

Some task names have changed:

- `docs:build` -> `website:check`
- `docs:serve` -> `website:serve`
@codecov-commenter
Copy link

Codecov Report

Merging #108 (e1cf7fa) into main (3222d17) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #108   +/-   ##
=======================================
  Coverage   11.49%   11.49%           
=======================================
  Files          18       18           
  Lines        1722     1722           
=======================================
  Hits          198      198           
  Misses       1466     1466           
  Partials       58       58           
Flag Coverage Δ
unit 11.49% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3222d17...e1cf7fa. Read the comment docs.

@per1234 per1234 merged commit b70ce14 into arduino:main Aug 16, 2021
@per1234 per1234 deleted the check-website branch August 16, 2021 12:02
@per1234 per1234 self-assigned this Nov 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants