Skip to content

ci: use lint action from meta + add lint-sync - #499

Merged
caarlos0 merged 2 commits into
mainfrom
update-workflows
Oct 2, 2025
Merged

caarlos0 merged 2 commits into
mainfrom
update-workflows

Conversation

@andreynering

Copy link
Copy Markdown
Member

Went checking because of #498, and realized our actions were out-of-date.

@andreynering andreynering self-assigned this Oct 2, 2025
@andreynering
andreynering requested a review from a team as a code owner October 2, 2025 13:01
@andreynering
andreynering requested review from aymanbagabas and kujtimiihoxha and removed request for a team October 2, 2025 13:01
secrets:
gh_pat: ${{ secrets.PERSONAL_ACCESS_TOKEN }}

codecov:

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I was in doubt on what to do for this one. I kept as is for now.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think our workflow on meta does the codecov stuff too

@andreynering andreynering Oct 2, 2025 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The build workflow doesn't, apparently.

We have a separate coverage workflow there, but I haven't found a repo that uses that. Lip Gloss and Bubble Tea have their own coverage workflow.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

oh, yeah, true... so this is prob good to merge

@andreynering
andreynering requested a review from caarlos0 October 2, 2025 13:01
@andreynering andreynering mentioned this pull request Oct 2, 2025
2 tasks done
@caarlos0
caarlos0 merged commit 6b4494c into main Oct 2, 2025
@caarlos0
caarlos0 deleted the update-workflows branch October 2, 2025 18:55
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

⌂ 换网址