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

docs: Update the contribution guidelines (no-changelog) #5165

Merged
merged 1 commit into from
Jan 19, 2023

Conversation

svzi
Copy link
Contributor

@svzi svzi commented Jan 15, 2023

Whilst setting up my mac for development purposes, I've stumbled upon a few difficulties in the CONTRIBUTING.mg guide. So I thought to make some changes to the requirements section. For example the NodeJS and pnpn versions that are defined in the engines section of the package.json. I've also updated some other parts and re-sorted two section. This way a new contributor should be able to follow the guide step by step and getting the understanding of what he does. That was missing during my setup. I hope it's now more clear to new developer, joining the project.

If you want me to change or revert something, please let me know.

Best,
Sven

@CLAassistant
Copy link

CLAassistant commented Jan 15, 2023

CLA assistant check
All committers have signed the CLA.

@n8n-assistant n8n-assistant bot added the community Authored by a community member label Jan 15, 2023
@svzi svzi changed the title docs: Updated the contribution guidelines docs: Update the contribution guidelines Jan 16, 2023
@Joffcom Joffcom changed the title docs: Update the contribution guidelines docs: Update the contribution guidelines (no-changelog) Jan 19, 2023
@Joffcom Joffcom merged commit 4ebf40c into n8n-io:master Jan 19, 2023
@Joffcom
Copy link
Member

Joffcom commented Jan 19, 2023

Hey @svzi,

Thanks for the PR, This has now been merged 🎉

@svzi svzi deleted the improved-contributing-guide branch January 19, 2023 09:46
@n8n-assistant n8n-assistant bot added the Upcoming Release Will be part of the upcoming release label Jan 19, 2023
MiloradFilipovic added a commit that referenced this pull request Jan 19, 2023
* master:
  docs: Update the contribution guidelines (no-changelog) (#5165)
  fix(HTTP Request Node): Bug - node requires string instead of json
  fix(GitLab Node): Update credential test endpoint (#5166)
  fix(editor): Allow special chars in node selector completion (#5196)
  test: Add tests for pinning data (#5157)
  fix(core): Fix migration (no-changelog) (#5186)
  fix(HTTP Request Node): Response format to text is ignored for JSON responses
  fix(Gmail Trigger Node): Filter by labels not working (#5173)
  feat(Item Lists Node): Table tranformation
  📚 Update CHANGELOG.md and main package.json to 0.211.2
  🔖 Release [email protected]
  ⬆️ Set [email protected] on n8n
  🔖 Release [email protected]
  fix(core): Restore community nodes installation (#5180)
  feat: (Google Sheets Trigger Node): Trigger for Google Sheets
  test: Add e2e tests for PRs (#5176)
@janober
Copy link
Member

janober commented Jan 19, 2023

Got released with [email protected]

@janober janober removed the Upcoming Release Will be part of the upcoming release label Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Authored by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants