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

chore: remove installation details in README.md #344

Merged
merged 1 commit into from
Dec 8, 2022

Conversation

stijnmoreels
Copy link
Member

The README.md file contained duplicated installation details from the documentation. It is rather hard to keep this up-to-date and the only added-value would be that people who didn't via our documentation site know how certain modules have to be installed. This information is already available on our documentation site and we would rather guide people to the documentation than to our GitHub root path, I guess.

@netlify
Copy link

netlify bot commented Dec 7, 2022

Deploy Preview for arcus-scripting canceled.

Name Link
🔨 Latest commit 024dd9e
🔍 Latest deploy log https://app.netlify.com/sites/arcus-scripting/deploys/639033e43213ac0009b5b95d

@pim-simons
Copy link
Contributor

✅ Deploy Preview for arcus-scripting canceled.

Name Link
🔨 Latest commit 024dd9e
🔍 Latest deploy log app.netlify.com/sites/arcus-scripting/deploys/639033e43213ac0009b5b95d

@stijnmoreels this deploy seems to have failed, any idea why? Log seems empty.

@stijnmoreels
Copy link
Member Author

@stijnmoreels this deploy seems to have failed, any idea why? Log seems empty.

I think it's bc there are no changes in the source. The logs also has this:

7:34:36 AM: No changes detected in base directory. Returning early from build.

So there is no change in deploy too.

@stijnmoreels stijnmoreels merged commit a6bdb50 into main Dec 8, 2022
@stijnmoreels stijnmoreels deleted the chore/remove-installation-details branch December 8, 2022 05:44
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.

2 participants