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

Follow Up of PR#11 in bookbrainz-dev-docs #984

Merged
merged 2 commits into from May 4, 2023

Conversation

Shelly011s
Copy link
Contributor

Problem

This PR is a follow up of PR in bookbrainz-dev-docs.

Solution

I have already copied the content of the deleted files in the PR.

Areas of Impact

I have deleted DEPENDENCIES_MANUAL_INSTALL.md, INSTALLATION_TROUBLESHOOTING.md, NODEJS_SETUP.md to prevent redundancy and moved/copied documents in the bookbrainz-dev-docs.

@Shelly011s Shelly011s changed the title removed redundant md files Follow Up of PR#11 in bookbrainz-dev-docs Apr 24, 2023
Copy link
Contributor

@MonkeyDo MonkeyDo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for opening a PR @Shelly011s !

I can't seem to find the content of CONTRIBUTING.md in the developer docs, and I think generally it is probably one file we do want to keep on the repo side (it's a special sort of file, see here for some details )

So I'll ask to put that file back in. The other ones are fine on the developer docs, and now we won't have possible duplication :)

Copy link
Contributor

@MonkeyDo MonkeyDo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you !
🚀

@MonkeyDo MonkeyDo merged commit c3f7b19 into metabrainz:master May 4, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants