Skip to content

chore: Fix vale linter#19372

Merged
kodiakhq[bot] merged 3 commits intomainfrom
chore/fix_vale
Oct 14, 2024
Merged

chore: Fix vale linter#19372
kodiakhq[bot] merged 3 commits intomainfrom
chore/fix_vale

Conversation

@erezrokah
Copy link
Copy Markdown
Member

@erezrokah erezrokah commented Oct 14, 2024

@cq-bot cq-bot added the area/ci label Oct 14, 2024
steps:
- name: Git checkout
uses: actions/checkout@v4
- run: |
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.

echo "$venv/bin" >> $GITHUB_PATH
- name: Vale
uses: errata-ai/vale-action@38bf078c328061f59879b347ca344a718a736018
uses: errata-ai/vale-action@3a4769d4af5f9498e586fb41abc72bbca594b400
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.

Updated to latest version while we're at it

@erezrokah erezrokah changed the title chore: Update vale linter chore: Fix vale linter Oct 14, 2024
@erezrokah erezrokah marked this pull request as ready for review October 14, 2024 10:05
@erezrokah erezrokah requested review from a team and marianogappa and removed request for a team October 14, 2024 10:05
@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Oct 14, 2024
@kodiakhq kodiakhq bot merged commit f33b75a into main Oct 14, 2024
@kodiakhq kodiakhq bot deleted the chore/fix_vale branch October 14, 2024 11:37
erezrokah added a commit to cloudquery/plugin-sdk that referenced this pull request Oct 15, 2024
#### Summary

Same as cloudquery/cloudquery#19372

---

Use the following steps to ensure your PR is ready to be reviewed

- [ ] Read the [contribution guidelines](../blob/main/CONTRIBUTING.md)
🧑‍🎓
- [ ] Run `go fmt` to format your code 🖊
- [ ] Lint your changes via `golangci-lint run` 🚨 (install golangci-lint
[here](https://golangci-lint.run/usage/install/#local-installation))
- [ ] Update or add tests 🧪
- [ ] Ensure the status checks below are successful ✅
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants