Skip to content
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

update k6lint to v0.2.0 #99

Merged
merged 3 commits into from
Nov 8, 2024
Merged

update k6lint to v0.2.0 #99

merged 3 commits into from
Nov 8, 2024

Conversation

szkiba
Copy link
Collaborator

@szkiba szkiba commented Nov 8, 2024

This is an internal maintenance release.

  • Update k6lint to v0.2.0
  • Compliance check TTL

k6lint v0.2.0

k6lint v0.2.0 includes two new checkers:

  • build checker
  • smoke checker

build checker

The check is successful if the extension can be built with the latest k6 release.

smoke checker

The check is successful if there is a smoke test script and it runs successfully with the k6 built with the extension.

Obviously, a prerequisite for a successful run is that the build checker runs successfully, otherwise k6 cannot be built with the extension.

The smoke test script file is searched for in the root of the repository and in the test,tests,examples directories. The name of the smoke test script is one of the following:

  • smoke.js
  • smoke.ts
  • smoke.test.js
  • smoke.test.ts

Compliance check TTL

The result of the compliance check was previously cached until the source code of the given extension was changed.

The result of the build check depends not only on the source code of the given extension, but also on the latest k6 release. This made it necessary to introduce a TTL (time to live) value in the compliance check cache.

@szkiba szkiba requested a review from a team as a code owner November 8, 2024 13:35
@szkiba szkiba requested review from pablochacin and removed request for a team November 8, 2024 13:35
@szkiba szkiba linked an issue Nov 8, 2024 that may be closed by this pull request
@szkiba szkiba removed the request for review from pablochacin November 8, 2024 13:35
@szkiba szkiba merged commit da01813 into main Nov 8, 2024
5 checks passed
@szkiba szkiba deleted the 98-update-k6lint-to-v0.2.0 branch November 8, 2024 13:43
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.

Update k6lint to v0.2.0
1 participant