Skip to content

fix: Better build overrides#1492

Merged
kodiakhq[bot] merged 3 commits into
mainfrom
fix/linking-overrides
Jan 29, 2024
Merged

fix: Better build overrides#1492
kodiakhq[bot] merged 3 commits into
mainfrom
fix/linking-overrides

Conversation

@candiduslynx
Copy link
Copy Markdown
Contributor

Instead of passing static option we should allow for env var values overrides in the package call.

Comment thread plugin/plugin_package.go Outdated
@candiduslynx candiduslynx requested a review from disq January 29, 2024 13:02
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 29, 2024

⏱️ Benchmark results

Comparing with d177320

  • Glob-8 ns/op: 90.51 ⬆️ 0.06% increase vs. d177320

Copy link
Copy Markdown
Member

@disq disq left a comment

Choose a reason for hiding this comment

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

How about adding Tags to BuildTarget?

@candiduslynx
Copy link
Copy Markdown
Contributor Author

How about adding Tags to BuildTarget?

I think we'll add them when we'll really need them 😃

@kodiakhq kodiakhq Bot merged commit ca5afc1 into main Jan 29, 2024
@kodiakhq kodiakhq Bot deleted the fix/linking-overrides branch January 29, 2024 15:00
kodiakhq Bot pushed a commit that referenced this pull request Jan 29, 2024
🤖 I have created a release *beep* *boop*
---


## [4.27.2](v4.27.1...v4.27.2) (2024-01-29)


### Bug Fixes

* Better build overrides ([#1492](#1492)) ([ca5afc1](ca5afc1))
* When `_cq_id` SyncMigrateMessage not sent ([#1489](#1489)) ([d177320](d177320))

---
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