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: add runPubGetOffline option #326

Merged
merged 1 commit into from
Jun 21, 2022

Conversation

mspb1g12
Copy link
Contributor

Description

Add an option to run all pub gets offline, relying on pre-populated pub caches.
This is useful in our CI environment which has a locked down network. Ideally the --offline flag (and other pub get flags) should be passed from bootstrap through to pub get but this seems like the simpler change in the short run, though if that seems preferable I can look into how it might work.
Another alternative could be runPubGetOfflineInCI using https://pub.dev/packages/ci to detect the CI environment which would be more useful but may be too specific a use-case.

Type of Change

  • feat -- New feature (non-breaking change which adds functionality)
  • 🛠️ fix -- Bug fix (non-breaking change which fixes an issue)
  • ! -- Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 refactor -- Code refactor
  • ci -- Build configuration change
  • 📝 docs -- Documentation
  • 🗑️ chore -- Chore

@CLAassistant
Copy link

CLAassistant commented Jun 18, 2022

CLA assistant check
All committers have signed the CLA.

@mspb1g12 mspb1g12 force-pushed the feat/offline-pub-get branch 2 times, most recently from 8018cb4 to 7065a5d Compare June 18, 2022 15:31
@mspb1g12 mspb1g12 force-pushed the feat/offline-pub-get branch from 7065a5d to 837daaa Compare June 18, 2022 15:43
Copy link
Collaborator

@blaugold blaugold 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 contributing! LGTM

@blaugold blaugold requested a review from Salakar June 20, 2022 07:26
@blaugold blaugold merged commit 8358a5a into invertase:main Jun 21, 2022
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.

4 participants