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

Add Foods for Tomorrow / Heura #57

Merged
merged 1 commit into from
Dec 26, 2019
Merged

Conversation

aalises
Copy link
Contributor

@aalises aalises commented Dec 23, 2019

Checklist

Reviewed MRs:

  • added apeel sciences #53

  • Add WMO and IGRA #56

  • I reviewed 2 other open pull requests. Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration.

  • I read and followed the CONTRIBUTING guidelines.

  • This pull request has a title in the format Add Company Name. Or Add Company1, Company2 and Company3 if it adds multiple companies.

    • ❌ Update README.md
    • ❌ Added Orbital Systems
    • ✅ Add Orbital Systems
    • ✅ Add Orbital Systems and Bulb
  • I added entries in alphabetical order.

  • The description is simple and doesn't contain unnecessary buzzwords. It needs to say what the company does, it's not a sales pitch.

    • ❌ Disruptive company developing cutting-edge AI-enabled solutions to provide farmers with a decision support system.
    • ✅ They use satellites to help farmers monitor crops and lower their emissions by reducing their nitrogen usage.

@aalises aalises force-pushed the aalises/add-heura branch 2 times, most recently from 41da012 to f9ba610 Compare December 23, 2019 15:19
@nglgzz
Copy link
Owner

nglgzz commented Dec 26, 2019

Hey @aalises, thank you for your contribution, this seems like a great addition to the list!

Only one thing, I would replace the | in Foods for Tomorrow | Heura with some other separator, as currently that is used to separate between different types of information.

@aalises
Copy link
Contributor Author

aalises commented Dec 26, 2019

Sure @nglgzz ! Changed ^^

@nglgzz nglgzz merged commit 6498440 into nglgzz:master Dec 26, 2019
@nglgzz
Copy link
Owner

nglgzz commented Dec 26, 2019

Awesome, thank you again for your contribution ✨

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