Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

setup: remove 404 link, reference to Dockerfile #859

Merged
merged 1 commit into from
Aug 26, 2019
Merged

Conversation

julianoes
Copy link
Contributor

This link was no longer valid, the Dockerfile is more helpful.

This link was no longer valid, the Dockerfile is more helpful.
@julianoes
Copy link
Contributor Author

Also see: PX4/PX4-containers#195 and #798.

@hamishwillee
Copy link
Collaborator

Thanks very much @julianoes - merging.

Should we remove all the "manual instructions" too? (except the group permissions, which you can't infer from docker)

@hamishwillee hamishwillee merged commit 8456e74 into master Aug 26, 2019
@hamishwillee hamishwillee deleted the pr-arch branch August 26, 2019 22:10
@julianoes
Copy link
Contributor Author

Should we remove all the "manual instructions" too?

I want to follow up with Arch Linux, I'll do it then.

@hamishwillee
Copy link
Collaborator

Thanks. Sounds excellent.

What I'd quite like is install scripts that are self documenting, and run in CI - ie run to setup dev environment AND that builds are done for those environments. Then we could reduce the manual setup instructions to just what can't be automated.

@julianoes
Copy link
Contributor Author

Yes, exactly!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants