Skip to content

chore: Clean up CI#543

Merged
erezrokah merged 1 commit intomainfrom
chore/clean_up_ci
Dec 29, 2022
Merged

chore: Clean up CI#543
erezrokah merged 1 commit intomainfrom
chore/clean_up_ci

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Summary

This aligns the SDK CI with other repositories


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

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests 🧪
  • Ensure the status checks below are successful ✅

uses: errata-ai/vale-action@v2
with:
vale_flags: "--glob='!{plugins/.snapshots/*,CHANGELOG.md,.github/styles/proselint/README.md}'"
vale_flags: "--glob=!{plugins/.snapshots/*,CHANGELOG.md,.github/styles/proselint/README.md}"
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.

Vale doesn't work with the '' quotes

package-name: plugin-sdk
token: ${{ secrets.GH_CQ_BOT }}
pull-request-title-pattern: "chore${scope}: Release${component} v${version}"
bump-minor-pre-major: true
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.

We don't need this anymore as we released our major version

@github-actions
Copy link
Copy Markdown

⏱️ Benchmark results

Comparing with a6a6a8e

  • DefaultConcurrency-2 resources/s: 12,227 ⬆️ 3.51% increase vs. a6a6a8e
  • Glob-2 ns/op: 199.6 ⬆️ 1.55% increase vs. a6a6a8e
  • TablesWithChildrenDefaultConcurrency-2 resources/s: 28,127 ⬇️ 6.69% decrease vs. a6a6a8e
  • BufferedScanner-2 ns/op: 11.68 ⬇️ 9.67% decrease vs. a6a6a8e
  • LogReader-2 ns/op: 37.03 ⬇️ 0.65% decrease vs. a6a6a8e

@erezrokah erezrokah merged commit dcd8834 into main Dec 29, 2022
@erezrokah erezrokah deleted the chore/clean_up_ci branch December 29, 2022 12:31
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