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

feat: support absolute path for options that makes sense #1360

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ANGkeith
Copy link
Collaborator

@ANGkeith ANGkeith commented Oct 3, 2024

resolves #1067

with this implementation,

we should be able to prevent gcl's stateDir from polluting the repo with

export GCL_STATE_DIR=$XDG_STATE_HOME/gitlab-ci-local

Breaking changes




  • @avoidik could you help to test out this changes to see if there's any bug ? in case i missed out something

@ANGkeith ANGkeith marked this pull request as draft October 3, 2024 05:01
@ANGkeith ANGkeith force-pushed the feat/decouple-from-cwd branch 4 times, most recently from 85e258e to 4ec2036 Compare October 3, 2024 06:37
@ANGkeith ANGkeith marked this pull request as ready for review October 3, 2024 06:41
@ANGkeith ANGkeith changed the title feat: add support for absolute path for options that makes sense feat: add absolute path support for options that makes sense Oct 3, 2024
@ANGkeith ANGkeith changed the title feat: add absolute path support for options that makes sense feat: support absolute path for options that makes sense Oct 3, 2024
@firecow
Copy link
Owner

firecow commented Oct 7, 2024

@ANGkeith Should we put this on the 5.x.x roadmap? Or do you think developers can handle it in a 4.x.x release?

@ANGkeith
Copy link
Collaborator Author

ANGkeith commented Oct 8, 2024

Should we put this on the 5.x.x roadmap?

ok let's go with this.. don't want to make any enemies... haha

possible to make a major release soon ?

This mr touches too much file.. don't really want to leave it lying around for too long haha

@firecow
Copy link
Owner

firecow commented Oct 8, 2024

Yeah, I'm creating a 5.x.x branch, lets work on that the next couple of weeks.
It's about time, we get some sane defaults set 😄

@ANGkeith ANGkeith marked this pull request as draft October 9, 2024 16:26
Copy link

sonarcloud bot commented Oct 11, 2024

@ANGkeith ANGkeith added the 5.x.x label Oct 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Decouple from cwd
2 participants