When you are refactoring code there will be times you run across code that cannot be easily changed or may be better suited for an alternate branch. In this case you can
- Create a follow-up issue
- Document the code by a comment as shown below
/* @@ TODO comment about what needs to be done and/or why it is a problem, see [link to issue]
Comment blocks look:
/**
* comments
* more comments
*/One-line comments look like:
// comments
doSomething()
*/In any given branch/PR there should be no more than a couple files modified. Try to stick to one, however for tests you may need two, one for the test and one for the code you are testing.
Name branch..
Add WIP in the title of the PR if you want review..
TypeScript Comments Test Examples
- Rename file to .ts
- Add types to public methods (if this is dificult you can add the 'any' type and add comments as described above to indicate it needs further work)
- Add comments
- No Logic changes, only minor refactoring
- PR is reviewed by one other engineer and merged to Master
- Write unit tests
- Write examples
- Refactor
Notes: can't use single quotes in scripts. Need to use `" instead Need to run builds using bash because of sh command.