Skip to content

Lint in docker#717

Closed
wakamex wants to merge 42 commits intomainfrom
lint_in_docker_tags_unchanged
Closed

Lint in docker#717
wakamex wants to merge 42 commits intomainfrom
lint_in_docker_tags_unchanged

Conversation

@wakamex
Copy link
Copy Markdown
Contributor

@wakamex wakamex commented Jul 21, 2023

  • image is smaller (1.06GB from 1.16GB)
  • workflows are faster:
  • lint in 33s down from 1m 13s
  • static in 52s down from 1m 40s
  • test in 56s down from 2m 2s
  • no change to docker tags

@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 21, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
elf-simulations ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 23, 2023 1:24am

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4897 🚀

@wakamex wakamex changed the title Lint in docker tags unchanged Lint in docker Jul 21, 2023
@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4982 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4972 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4952 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4874 🚀

@wakamex
Copy link
Copy Markdown
Contributor Author

wakamex commented Jul 21, 2023

this passes with an image built in the new way (which has coverage installed)
tested with images build that way, tagged with the "testing" tag in #703

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4979 🚀

@wakamex wakamex marked this pull request as draft July 22, 2023 04:59
@wakamex
Copy link
Copy Markdown
Contributor Author

wakamex commented Jul 22, 2023

will update after #727

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4999 🚀

@wakamex wakamex closed this Aug 8, 2023
@wakamex wakamex deleted the lint_in_docker_tags_unchanged branch August 8, 2023 17:56
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