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

Issue 639 #816

Merged
merged 9 commits into from
Oct 19, 2020
Merged

Issue 639 #816

merged 9 commits into from
Oct 19, 2020

Conversation

strawberries73
Copy link
Contributor

This is a modified version of the Contributing Guidelines.

This pull request adheres to the repository's Code of Conduct.

  • I am an employee of the company mentioned and confirm all included details are correct
  • This PR contains housekeeping only (URL edits, copy changes etc)
  • You know your alphabet - company is listed in alphabetical order in the README
  • The company directly hires employees. No bootcamps / freelance sites / etc
  • The company hires remote employees, or positions are available to remote workers and are clearly illustrated as such
  • A company profile is included - Required for new additions. (This can be a basic outline but at least something please)
  • Remote status has details regarding how the culture includes remote employees, how the company integrated remote workers, etc
  • Region details the geographic regions in which this company's employees can reside. For more details see the instructions in the example company profile.
  • How to apply details the best approach for new applications, page on site where open position are listed, and any other help available for job hunters

@auto-nylen
Copy link

Hi @strawberries73, thanks for your contribution!

Our automated checks detected some problems with this pull request (as of commit 7cfa7f8). See below for details:

black-pixel.md: Profile looks complete, but the main readme contains a warning emoji.

1 problem detected

Please fix these issues and push a new commit to this pull request.

@auto-nylen
Copy link

Hi @strawberries73, thanks for updating the pull request! According to our automated checks, everything looks OK now (as of commit c9a065a). ✔️

@dougaitken
Copy link
Member

Hey @strawberries73 can I ask you to rename this to the new company and then delete the old company from the list and company profile file, please?

@strawberries73
Copy link
Contributor Author

Yes. I will do that.

@strawberries73
Copy link
Contributor Author

Hi @dougaitken, I added the new company and deleted the old one as requested. The README is also updated.

@dougaitken
Copy link
Member

Did you commit and push your changes @strawberries73? I don't see any new commits to this PR

@dougaitken
Copy link
Member

I fixed it up @strawberries73 - thanks for your edit!

@strawberries73
Copy link
Contributor Author

Thank you, I didn't realize that I pushed it to my forked repository. I'm new and still learning the steps. I appreciate that you fixed it.

@dougaitken
Copy link
Member

Not a problem @strawberries73 happy to help get people started!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants