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

update gitpod conf to main branch? #1680

Merged
merged 2 commits into from
Jul 5, 2020
Merged

update gitpod conf to main branch? #1680

merged 2 commits into from
Jul 5, 2020

Conversation

jywarren
Copy link
Member

@jywarren jywarren commented Jul 5, 2020

If this passes, we might also set up the default test in GitPod to add all modules, and a smaller image, so we get to see it run through all modules by default!

@gitpod-io
Copy link

gitpod-io bot commented Jul 5, 2020

@codecov
Copy link

codecov bot commented Jul 5, 2020

Codecov Report

Merging #1680 into main will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1680   +/-   ##
=======================================
  Coverage   65.42%   65.42%           
=======================================
  Files         132      132           
  Lines        2742     2742           
  Branches      438      438           
=======================================
  Hits         1794     1794           
  Misses        948      948           

@jywarren
Copy link
Member Author

jywarren commented Jul 5, 2020

Hmm. @nisarhassan12 do you think this was wrong? I tried to ask a clarifying question in gitpod-io/gitpod#1646 but am not sure if i should revert this. Thanks for your help!

@jywarren
Copy link
Member Author

jywarren commented Jul 5, 2020

Oops misspelled @nisarhassan12 apologies. Corrected above, thank you!

jywarren added a commit that referenced this pull request Jul 6, 2020
@jywarren
Copy link
Member Author

jywarren commented Jul 6, 2020

Undid this! ef3afc0 following suggestions from gitpod-io/gitpod#1646

I just removed the whole line bc prebuilds are on by default.

Thanks!

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.

None yet

1 participant