Skip to content

Conversation

@hermanschaaf
Copy link
Member

I noticed we use the -dir suffix fairly consistently in the CloudQuery CLI, so figured we should keep it the same here.

@hermanschaaf hermanschaaf changed the title fix: Use -dir suffix for arguments fix: Use -dir suffix for plugin package arguments Sep 13, 2023
@github-actions github-actions bot added fix and removed fix labels Sep 13, 2023
@github-actions
Copy link

⏱️ Benchmark results

Comparing with 393c94d

  • Glob-8 ns/op: 99.59 ⬇️ 1.52% decrease vs. 393c94d

@codecov
Copy link

codecov bot commented Sep 13, 2023

Codecov Report

Patch coverage: 66.66% and no project coverage change.

Comparison is base (393c94d) 48.51% compared to head (5190dd1) 48.51%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1213   +/-   ##
=======================================
  Coverage   48.51%   48.51%           
=======================================
  Files          87       87           
  Lines        8142     8142           
=======================================
  Hits         3950     3950           
  Misses       3828     3828           
  Partials      364      364           
Files Changed Coverage Δ
serve/package.go 55.85% <66.66%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kodiakhq kodiakhq bot merged commit 93f9398 into main Sep 13, 2023
@kodiakhq kodiakhq bot deleted the use-dir branch September 13, 2023 14:30
kodiakhq bot pushed a commit that referenced this pull request Sep 19, 2023
🤖 I have created a release *beep* *boop*
---


## [4.8.0](v4.7.1...v4.8.0) (2023-09-19)


### Features

* Add Checksums to package.json format ([#1217](#1217)) ([720baae](720baae))
* Add message to package command ([#1216](#1216)) ([44956d9](44956d9))
* Add shuffle scheduler ([#1218](#1218)) ([2b1ba30](2b1ba30))
* Update package command ([#1211](#1211)) ([39fc65e](39fc65e))


### Bug Fixes

* Add schema version to package.json ([#1212](#1212)) ([393c94d](393c94d))
* **deps:** Update github.com/cloudquery/arrow/go/v14 digest to 483f6b2 ([#1209](#1209)) ([179769a](179769a))
* **deps:** Update github.com/cloudquery/arrow/go/v14 digest to ffb7089 ([#1215](#1215)) ([70f20bb](70f20bb))
* Use -dir suffix for plugin package arguments ([#1213](#1213)) ([93f9398](93f9398))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants