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

PhishMe rebranded as Cofense #401

Merged
merged 2 commits into from
Mar 20, 2018
Merged

Conversation

movermeyer
Copy link
Contributor

@movermeyer movermeyer commented Mar 18, 2018

See press release

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 added 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)
  • Any missing parts from the profile, please use <Unknown> to allow for future completion
  • Remote status has details regarding how the culture includes remote employees, how the company integrated remote workers, etc
  • Region details any restrictions to applicants based on geography
  • 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 @movermeyer, thanks for your contribution!

Our automated checks detected some problems with your pull request (as of commit 44c1d4d). See below for details:

README.md: Expected 3 table cells but found 2: <td>Cofense</td><td><a href="https://cofense.com">https://cofense.com</a></td>

1 problem detected

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

@dougaitken
Copy link
Member

Hi @movermeyer, are you able to submit a company profile as well, please? Adding a timezone will also help on the readme

@auto-nylen
Copy link

Hi @movermeyer, thanks for updating the pull request! According to our automated checks, everything looks OK now (as of commit eb3bd90). ✅

@movermeyer
Copy link
Contributor Author

I don't work there, but I did some digging on their site and made a profile.

@dougaitken
Copy link
Member

Fantastic, thanks @movermeyer. I collated most of the original list, most of it was pure digging on company websites :)

The top check in the list is just something for me to gauge interest

@dougaitken dougaitken merged commit 9106cc1 into remoteintech:master Mar 20, 2018
@nylen nylen mentioned this pull request Apr 14, 2018
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.

None yet

3 participants