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

Add EARTHLY_CI arg #2398

Merged
merged 4 commits into from Nov 21, 2022
Merged

Add EARTHLY_CI arg #2398

merged 4 commits into from Nov 21, 2022

Conversation

miles170
Copy link
Contributor

Closes #2261.

This adds an EARTHLY_CI builtin arg which is set to true when earthly is run with the --ci flag.

This adds an `EARTHLY_CI` builtin arg which is set to `true` when
earthly is run with the `--ci` flag.
@CLAassistant
Copy link

CLAassistant commented Nov 11, 2022

CLA assistant check
All committers have signed the CLA.

tests/ci-arg.earth Outdated Show resolved Hide resolved
tests/ci-arg.earth Outdated Show resolved Hide resolved
Copy link
Collaborator

@alexcb alexcb left a comment

Choose a reason for hiding this comment

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

thanks for the PR, looks good -- just left a couple minor requests.

@miles170 miles170 requested a review from alexcb November 18, 2022 16:00
* main: (30 commits)
  Support NETRC environmental var (earthly#2426)
  Add installation name to the changelog (earthly#2427)
  Add some basic buffering to conslogging (earthly#2425)
  switch satellite back to core-test (earthly#2424)
  Don't expire when JWT set externally (earthly#2418)
  Add all the non-buildkit output to logstream too (earthly#2423)
  More fields on satellite ls and inspect (earthly#2412)
  Fix linebreak (earthly#2421)
  Fix logstream closing sequence (earthly#2422)
  fix singlequote parsing for --shell-out-anywhere (earthly#2419)
  Update earthfile-syntax-highlighting release instructions (earthly#2420)
  Changes in vscode-syntax-highlighting-v0.0.12 (earthly#2407)
  fix incorrect session ID passed to llb.SessionID (earthly#2413)
  Allow passing a JWT to Earthly for accessing cloud features (earthly#2414)
  Fix `time.Now().UnixNano()` stripping away the nanos (earthly#2415)
  Update CHANGELOG.md (earthly#2393)
  docs: Satellite sleep behaviour (earthly#2380)
  Bump nokogiri from 1.12.5 to 1.13.9 in /examples/ruby-on-rails (earthly#2411)
  Bump nokogiri from 1.11.0 to 1.12.5 in /examples/ruby-on-rails (earthly#2295)
  run tests on docs (earthly#2410)
  ...
Signed-off-by: Alex Couture-Beil <alex@earthly.dev>
@alexcb alexcb merged commit a49bcd7 into earthly:main Nov 21, 2022
@miles170 miles170 deleted the issue-2261-earthly-ci branch November 22, 2022 01:10
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.

Expose --ci status to targets
3 participants