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.

Sign up for GitHub

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

Jump to bottom

Suggestion: Remove the executors #2

Open
KyleTryon opened this issue Jan 21, 2021 · 1 comment
Open

Suggestion: Remove the executors #2

KyleTryon opened this issue Jan 21, 2021 · 1 comment
Labels
backlog Identified as a backlog item, often combined with low-priority and help-wanted labels enhancement New feature or request

Comments

@KyleTryon
Copy link
Contributor

Executors are useful for parameterized execution environments that will be reused. As this executor is used exclusively within the continue job, it should not be necessary and can sometimes be confusing for users.

Instead, I would recommend setting the docker executor and image statically on the continue job, including the same tag parameter for the image version tag.
@KyleTryon KyleTryon added the bug Something isn't working label Jan 21, 2021
@aaronfox1
Copy link

Made this update locally but need access to create patch-2 branch to create PR to merge to master.

@Fernando-Abreu Fernando-Abreu added enhancement New feature or request backlog Identified as a backlog item, often combined with low-priority and help-wanted labels and removed bug Something isn't working labels Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backlog Identified as a backlog item, often combined with low-priority and help-wanted labels enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Fernando-Abreu @KyleTryon @aaronfox1