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 gov.uk & skillsmap #3

Merged
merged 3 commits into from May 30, 2018
Merged

add gov.uk & skillsmap #3

merged 3 commits into from May 30, 2018

Conversation

@dmorgantini
Copy link
Contributor

@dmorgantini dmorgantini commented May 28, 2018

  • Add GOV.UK as they have published a very comprehensive (if pretty generic) set of career ladders
  • Add SkillsMap.io, the first company focused on adding structure around engineering career ladders
  • Remove ChartBeats as they no longer publish their engineering ladders.
* Add GOV.UK as they have published a very comprehensive (if pretty generic set of career ladders)
* Add SkillsMap.io, the first company focused on adding structure around engineering career ladders
* Remove ChartBeats as they no longer publish their engineering ladders.
@benjmin-r
Copy link
Owner

@benjmin-r benjmin-r commented May 29, 2018

Thanks a lot David, for your PR and the work you put in.
However, I'd prefer if the skillsmap entry was split in two, to present the Tes ladder with your video as one, and then create a separate section for the skillsmap tool. I'll probably do this next week or if you find the time and would like to do it before that, you're of course more than welcome to 😃
Also, if you make another change, be sure to add your name to the contributors section of the post. Except if you don't want your name there, which of course would also be fine.

@dmorgantini
Copy link
Contributor Author

@dmorgantini dmorgantini commented May 29, 2018

Certainly happy to do so. Will add a commit today.

dmorgantini and others added 2 commits May 29, 2018
.. also remove marketing-like description of Skillsmap
@benjmin-r
Copy link
Owner

@benjmin-r benjmin-r commented May 30, 2018

Thanks a lot again, David, for your work in extending the ladder list.
I restructured the Tes entry slightly and got rid of the last paragraph of the Skillsmap entry, as it didn't fit the style of the other entries, because it describes the offer of a company rather than a ladder (or the tool surrounding ladders).

@benjmin-r benjmin-r closed this May 30, 2018
@benjmin-r benjmin-r reopened this May 30, 2018
@benjmin-r benjmin-r merged commit 17dfeaf into benjmin-r:master May 30, 2018
1 check passed
1 check passed
deploy/netlify Deploy preview ready!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants