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 "Hiring from" column #102

Merged
merged 3 commits into from
Dec 9, 2019
Merged

Add "Hiring from" column #102

merged 3 commits into from
Dec 9, 2019

Conversation

lfilho
Copy link
Contributor

@lfilho lfilho commented Dec 7, 2019

Thanks for this repo. Here's a small contribution.

Purpose: Adds a "Hiring from" column. Fixes #40
Changes:

  • Fixed plain-text table formatting (realigning all misaligned | chars)
  • Added aforementioned column :-)

@yanirs
Copy link
Owner

yanirs commented Dec 8, 2019

Thanks for the contribution, @lfilho! It'd be a great addition, but to make it more useful, can you please populate the column a bit more? It'd be good to reduce the number of question marks. 🙂

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Company | Business/products | Tech stack/skills | Globally-competitive compensation? | Hiring From | Job page & other links
------------------------------------------------- | ----------------------------- | ---------------------------- | ------------------------------------ | ----------------------- | ----------------------
[10up](https://10up.com/) | Web consulting | WordPress, PHP, Node, Web | ✗ | ? | [Jobs](https://10up.com/careers/)<br>[Glassdoor](https://www.glassdoor.com.au/Overview/Working-at-10Up-EI_IE775906.11,15.htm)
[15Five](https://www.15five.com) | Continuous performance management suite | Django, Python, Javascript, React, Vue | ✓ | US | [Jobs](https://www.15five.com/careers)<br>[Glassdoor](https://www.glassdoor.com/Overview/Working-at-15Five-EI_IE1391015.11,17.htm)
Copy link
Owner

Choose a reason for hiding this comment

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

Interesting that it's US-only. I may end up removing single-country companies since the list is growing and I don't see such companies as being truly distributed.

@lfilho
Copy link
Contributor Author

lfilho commented Dec 8, 2019

Done. Added a dozen companies. Will do more as I keep learning and researching them eventually but hopefully this PR will make it easier for others to do the same

README.md Outdated
[Four Kitchens](http://www.fourkitchens.com/) | Web consulting | Drupal, Node, PHP, Web | ? | ? | [Jobs](http://www.fourkitchens.com/careers/)<br>[Glassdoor](https://www.glassdoor.com.au/Overview/Working-at-Four-Kitchens-EI_IE588769.11,24.htm)
[Genuitec](http://www.genuitec.com) | Eclipse-based tools | Java, Eclipse, Web, Mobile | ? | ? | [Jobs](http://www.genuitec.com/company/careers/)<br>[Glassdoor](https://www.glassdoor.com.au/Overview/Working-at-Genuitec-EI_IE346957.11,19.htm)
[Ghost](https://ghost.org/) | Publishing platform | Node, Web | ✓ | Anywhere | [Jobs](https://ghost.org/about/)
[GitHub](https://github.com) | GitHub | Git, Ruby, Web | ✓ | Mostly Anywhere | [Jobs](https://github.com/about/jobs)<br>[Glassdoor](https://www.glassdoor.com.au/Overview/Working-at-GitHub-EI_IE671945.11,17.htm)
Copy link
Owner

Choose a reason for hiding this comment

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

What does "mostly anywhere" mean? 🙂

Copy link
Contributor Author

Choose a reason for hiding this comment

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

If you check their website, they don't explicitly say they hire from anywhere but on the other hand they list Americas, Europe, Asia Pacific... Which cover a good chunk of the planet... :-)

Copy link
Owner

Choose a reason for hiding this comment

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

I think we can go with Anywhere then. 🙂

@yanirs
Copy link
Owner

yanirs commented Dec 8, 2019

Done. Added a dozen companies. Will do more as I keep learning and researching them eventually but hopefully this PR will make it easier for others to do the same

Thanks! It looks better now. I agree that having the new column would help with filling in the blanks. I'm happy to merge it once we clear up the "mostly anywhere" point.

@yanirs yanirs merged commit 07de02e into yanirs:master Dec 9, 2019
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.

Type of remote
2 participants