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

Revert pr121 arch fix #125

Merged
merged 2 commits into from
Mar 9, 2021
Merged

Revert pr121 arch fix #125

merged 2 commits into from
Mar 9, 2021

Conversation

eskaur
Copy link
Member

@eskaur eskaur commented Mar 7, 2021

Note how both commits in this PR are green (it now works with both archlinux/base and archlinux/archlinux:base), so the second commit is not strictly necessary. However, I added the second commit due to the discussion in #122 .
I'll let you Arch experts decide what's best.

@eskaur eskaur requested review from nedrebo and knzivid March 7, 2021 14:26
@eskaur eskaur requested a review from trym-b as a code owner March 7, 2021 14:26
Changing from archlinux/base to archlinux/archlinux:base, as discussed
in Issue #122.
@knzivid
Copy link
Contributor

knzivid commented Mar 8, 2021

Let's stick to arch's arch image

@knzivid
Copy link
Contributor

knzivid commented Mar 8, 2021

Like I previously mentioned, we are already rolling; why not roll with upstream?

@eskaur
Copy link
Member Author

eskaur commented Mar 9, 2021

Like I previously mentioned, we are already rolling; why not roll with upstream?

Guess I need approval from a "code owner". @nedrebo ?

@eskaur eskaur merged commit 780321d into master Mar 9, 2021
@eskaur eskaur deleted the revert-PR121-arch-fix branch March 9, 2021 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants