Skip to content

Latest commit

 

History

History
132 lines (94 loc) · 6.97 KB

CONTRIBUTING.md

File metadata and controls

132 lines (94 loc) · 6.97 KB

Bramble

Deployment

Anything merged into the branch bramble will be automatically deployed to staging. To deploy to production, push changes to the production branch.

The Basics

Filing a bug

Check the Troubleshooting Page for common issues with installing & launching Brackets, using Live Preview, etc.

For bugs be sure to search existing issues first. Include steps to consistently reproduce the problem, actual vs. expected results, and your OS and Brackets version number. Disable all extensions to verify the issue is a core Brackets bug. Read more guidelines for filing good bugs...

For feature requests please first check our feature backlog to see if it's already there. You can upvote features you'd like to see.

Submitting a pull request

Before you start coding, post to the brackets-dev Google group or the #brackets IRC channel on freenode about what you're thinking of working on, so you can get early feedback. We don't want you to do tons of work and then have to rewrite half of it!

For more on what's expected in a good pull request, see Contributing Code below.

Ways to Contribute

There are many ways you can contribute to the Brackets project:

Where Do I Start?

To start editing the Brackets code, read How to Hack on Brackets. To create your first Brackets extension, check out How to Write Extensions.

Here are some ideas:

  • Starter bugs can provide a good intro to the Brackets code.
  • Extension ideas are feature requests that we think would be best implemented as an add-on; it's up to the Brackets community to write them!
  • Starter features are a bit larger in scope. Be sure to discuss these in the newsgroup before starting. (To see starter features, click Filter Cards on the right and then click the green "Starter Feature" label).

Once you're ready to start coding, see the next section, Contributing Code.

I'm new to JavaScript. How can I contribute to Brackets? Brackets is a lot more complicated than the average website that uses JS. Better to start on some JS tutorials (like Codecademy's or MDN's) and contribute in some of the other ways listed above. Testing is a great way to start thinking like a programmer before you've learned to code!

Contributing Code

To get started editing the Brackets code, read How to Hack on Brackets.

Before submitting any pull request, please make sure to:

  1. Discuss any major changes or questions beforehand in the brackets-dev newsgroup.
  2. Consider whether your change would be better as an optional extension. Brackets is lightweight and tightly focused - but highly extensible.
  3. Follow the Pull Request Checklist to ensure a good-quality pull request.
  4. Sign the Brackets Contributor License Agreement (we cannot merge before this).

High quality code and a top-notch user experience are very important in Brackets, and we carefully review pull requests to keep it that way. The better you follow the guidelines above, the more likely we are to accept your pull request - and the faster the code review will go.

The Code Review Process

Brackets committers are responsible for reviewing all pull requests, providing feedback, and ultimately merging good code into master. The review process ensures all code is high quality, maintainable, and well documented.

Once you've opened a pull request, a committer will generally respond to it within a week with an initial set of comments (you don't need to ping anyone to find a reviewer). Some pull requests raise larger questions about UI design, product scope or architecture. Those are tagged to indicate that review will take longer:

  • [PM] - needs high-level input from product management
  • [XD] - needs UI design / visual design discussion
  • [ARCH] - needs architectural discussion

The best way to avoid this sort of holdup is to discuss your changes on the newsgroup first!

Once your pull request is merged, it will appear in the next release of Brackets - generally within two weeks.

Interested in becoming a committer? See the Committer Policy for details. Committers are expected to take a leading role in the project by making significant code contributions, reviewing pull requests, and providing feedback and suggestions on the direction of the project.

Even if you're not a committer, you're still welcome to give feedback on any pull request!

Adding New Files

When run in production (i.e., the resulting dist/ dir from running grunt build-browser-compressed), Bramble uses a Service Worker to cache and serve the app offline. In order to do this, the swPrecache grunt task generates a Service Worker ready to cache and serve all the necessary files. This file list is generated statically at build time based on the contents of the sw-cache-file-list.json file. If you add new files (e.g., a new default extension), make sure you add URL entries to this cache list.