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.

Sign up for GitHub

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

Jump to bottom

Set up continuous integration #2

Merged
merged 1 commit into from
Aug 22, 2020
Merged

Set up continuous integration #2

merged 1 commit into from
Aug 22, 2020

Conversation

felipecrs
Copy link
Contributor

No description provided.

@felipecrs
Copy link
Contributor Author

Any thoughts? Is it a good idea? It would help users to test their dotfiles somehow, and also helps us to test this template.

@felipecrs felipecrs marked this pull request as draft August 22, 2020 20:23
@felipecrs felipecrs force-pushed the felipecrs/actions branch 7 times, most recently from 4c65b3b to 1366f89 Compare August 22, 2020 20:54
@twpayne
Copy link
Member

twpayne commented Aug 22, 2020

This is a great idea, please merge :)

@felipecrs
Copy link
Contributor Author

Despite I'm still not able to run the container as a non-root user, it already works.

Ref: actions/checkout#211 (comment)

@felipecrs felipecrs marked this pull request as ready for review August 22, 2020 21:14
@felipecrs felipecrs merged commit 1d4e050 into master Aug 22, 2020
@felipecrs felipecrs deleted the felipecrs/actions branch August 22, 2020 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@felipecrs @twpayne