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

BLT-881: initial cut at developer skills documentation #902

Merged
merged 2 commits into from
Mar 14, 2017

Conversation

mikemadison13
Copy link
Contributor

Fixes #881 .

Changes proposed:

  • updating Onboarding doc to link to Skills Doc
  • Created Skills Doc that outlines minimum and suggested Skills for developers working with BLT

@mikemadison13
Copy link
Contributor Author

@grasmash and @danepowell I would love eyes on this to ensure I haven't left out any resources that are worthwhile, misstated any objectives, etc.

@danepowell
Copy link
Contributor

danepowell commented Jan 3, 2017

I like the idea of centralizing this kind of documentation, however I'm wondering if there's a less pointed way of linking to it from the onboarding doc. While it's true that developers should eventually gain all of these skills, they aren't absolutely necessary if all you want to do is install a local Drupal site or take BLT for a spin. I just don't want to make BLT seem unapproachable for newbies.

@mikemadison13
Copy link
Contributor Author

mikemadison13 commented Jan 3, 2017

@danepowell I agree that I don't want to scare people off, but I am trying to paint a realistic picture of "can you do these things?" if yes, then great. if no, then you should brush up or you'll have trouble. i'm happy to move other items into the recommended section if you think I've overinflated the "minimum" section.

readme/skills.md Outdated
## Enterprise Web Development
### Virtual Environment
#### [DrupalVM](https://www.drupalvm.com/)
This project aims to make spinning up a simple local Drupal test/development environment incredibly quick and easy, and to introduce new developers to the wonderful world of Drupal development on local virtual machines (instead of crufty old MAMP/WAMP-based development).
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't require that people use VM with BLT. In fact, I use MAMP :)

I'd remove this or note that it isn't required.

readme/skills.md Outdated
* [composer require](https://getcomposer.org/doc/03-cli.md#require)

####[NPM](https://www.npmjs.com/)
npm is the package manager for JavaScript. Find, share, and reuse packages of code from hundreds of thousands of developers — and assemble them in powerful new ways.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is not part of BLT. It is an aspect of Cog. We need to indicate that this isn't required. Perhaps a separate "optional" section.

readme/skills.md Outdated
* [Netbeans](http://netbeans.org/features/php/)

##Front End Technologies
###[Gulp](http://gulpjs.com/)
Copy link
Contributor

Choose a reason for hiding this comment

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

Likewise this is Cog specific.

readme/skills.md Outdated
* [git rebase -i](https://git-scm.com/docs/git-rebase#git-rebase---interactive)

### Github
GitHub is a web-based Git repository hosting service. It offers all of the distributed version control and source code management (SCM) functionality of Git as well as adding its own features. It provides access control and several collaboration features such as bug tracking, feature requests, task management, and wikis for every project.
Copy link
Contributor

Choose a reason for hiding this comment

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

GitHub isn't required by BLT.

@mikemadison13
Copy link
Contributor Author

@grasmash I have updated the language about a virtual environment (adding dev desktop, mentioning that it and drupal vm are optional). I also moved all of the NPM stuff / github down into the additional reading section (updated it to be optional) and added a note for all the front end stuff to reference cog and say it's not part of blt but is often bundled with it and its commands.

@grasmash grasmash merged commit 078cb69 into acquia:8.x Mar 14, 2017
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.

3 participants