* Some CLI updates and minor refactor
* Adding mock for author
* Adding --no-separators flag
* Rename to --remove-separators due to a commander issue
* Improved unit test
* Adding jsdoc
* Remove commented code
* Slice deployments for --top before making api requests
* More unit tests
* PR feedback
* More unit test improvements
* Left align everything but numbers, bring start time back
* Cluster sync
* Pull request id should be right aligned
* Right align duration, remove manual hld edit for now
* [HOUSEKEEPING] Finding broken links in md files and fix them
* using markdown-link-check
* update md-lint
* Update azure-pipelines.yml
* script is not needed because we do not support windows runtime
Co-authored-by: Yvonne Radsmikham <yvonne.radsmikham@gmail.com>
- Changed pipeline build/tests to support Node 12 (LTS) and 13 (current)
- Dropped Node 10
- Added Node 13
- Added ESLint pre-commit hook
- `eslint --fix` now runs for JS/TS files before being committed. If any
staged files fail linting, commit will fail.
- Added `eslint-config-prettier`
- Helps minimize conflicts between eslint and prettier configurations.
- Upgrade to TypeScript 3.8.
- No breaking changes.
- View changes here:
<https://devblogs.microsoft.com/typescript/announcing-typescript-3-8/>
- Upgrade to Jest 25
- No breaking changes.
- View changes here: <https://jestjs.io/blog/2020/01/21/jest-25>
- Upgrade to Prettier 2
- No Breaking changes.
- Default of `trailingComma`, `arrowParens`, and `endOfLine` have been
changed. Prettier has been run on the codebase to adjust for these changes.
- View changes here: <https://prettier.io/blog/2020/03/21/2.0.0.html>
- Removed TSLint and tslint.json configuration file
- Added ESLint with `recommended` TypeScript configurations.
- Removed all `I` prefix from type declarations (per ESLint recommendation)
- Have added return types to all function declarations (per ESLint
recommendation)
- To have minimal conflict with integration, many of the recommended ESLint
rules have been disabled on a per-page/line basis.
- We should attempt to remove these `/* eslint-disable ...` lines as we
refactor code and conform completely to the recommended linting rules.
* [FEATURE] Create resource group and ACR in spk setup command
* fix lint error
* fix lint error
* move azure container registry and resource group services to lib/azure folder
Co-authored-by: Andre Briggs <andrebriggs@users.noreply.github.com>
* Fix broken service build trigger
- Service pipeline build triggers were set to invalid value of `./` when the
service was in project root directory -- In such a case, we now longer include
the path trigger.
* removing async from fileutil test
Co-authored-by: Michael Tarng <20913254+mtarng@users.noreply.github.com>
Co-authored-by: Dennis Seah <dennis.seah@gmail.com>
* Refactored onboard commmand that closes#681
* Fixed lint issues
* Updated onboard doc
* Updated doc
* Fixed circualr call issue
* Fixed test cases and incorporated the feedback
* Fixed test case
* Fixed lint issues
* Added comments
* Updated output messages
* Updated doc
* Added validation for storage account and table names
* updated variable values in test cases
* Removed commented line of code
* Returned empty object when config yaml is undefined
* replaced echo with logger
* Removed new line breaks
* Pushing generate init
* linting
* Sourcing for infra generate
* Adding unit tests and mocks for infra generate source
* Updated tests for generate source
* master bump and unit test fix
* Updating docs
* adjusting simple-git
* refactoring infra generate per simple-git
* Upgrade spektate t version 0.1.11
* Add status for multi-stage pipelines
* Remove check in docker to hld pipeline
* Fix formatting
* Change to version 0.1.12
This PR focuses around ironing out the flow of running `spk project init` or
`spk project init -m`, creating pipelines via the generated
`azure-pipelines.yaml` files, and ensuring the pipeline can create appropriately
named/tagged docker images in ACR.
In order to allow for flexibility in how SRE's choose to deploy the pipelines,
this PR does not automate the actual deployment of the pipeline. Instead, `info`
level instructions have been added specifying how to run the
`spk service create-pipeline` to deploy the `azure-pipelines.yaml` generated; If
the the user has configured the spk config correctly, the outputted command is
copy/paste ready to deploy the generated pipelines yaml:
If ran on a mono-repo:
`spk service create-pipeline --project-dir packages my-service` will be logged.
If ran on a standard repo: `spk service create-pipeline my-service` will be
logged.
The `azure-pipelines.yaml` expects 4 environment variables to be present in
order to login to `az` cli and build and push an image to ACR:
- `ACR_NAME` - name of the target ACR
- `SP_APP_ID` - service principal ID
- `SP_PASS` - service principal password
- `SP_TENANT` - service principal tenant
These requirements are also `info` logged to the end user upon
`azure-pipelines.yaml` generation.
Previously `spk service create-pipeline` only worked for mono-repositories and
the code has been refactored such that the existence of the `--packages-dir`
options implies whether or not the repository is a mono-repository or a standard
one.
closes https://github.com/microsoft/bedrock/issues/646
Fixed an issue with the starter azure-pipelines.yaml an top level type
definition for it. `variables` expects a sequence of maps but a top
level map was being generated instead.
* Adding Variable group implementation
* added new environment variables in example file.
* refactoted code and added comments
* Changed node mode to development in webpack config and added debug statements in variable group
* Fixed naming issue in the production build. Added Service Point interfaces
* removed tslint disabling
* Fixed linting issues
* updated mock config file and extracted azure-devops interface
* Made some config variables nullable
* Updated mock config file with variablegroup info
* Added interfaces for service connection and implemented feedback comments
* Fixed lint issue
* renamed service connection to service endpoint to be consistent with rest api
* Fixed file reanming issue
* refatored azdo.ts methods to be singleton
* Fixed lint issues
* adding tests
* uncommented code
* Fixed unit test cases
* added more test cases and created new section for vsts variables
* added more test cases and created new section for vsts variables
* Added test cases
* updated test description
* updated comments
* refactored code and added create command for variable group
* added docs
* updated docs and added valiadtion to check variable group type
* Moved ts-node to dev dependencies
* updated docs
* renamed file
* Fixed type refactoring issue
* fixed doc error
* changed docs and fixed unit test
* Force merging master
* added variable group types back after force overwrite
* Dashboard command
* Basic tests for dashboard command
* Fix imports
* Improve tests
* Testing with a higher timeout
* Test simply won't end
* Don't open port in unit test
* Kill docker container before test finish
* Logger messages
* Try catch
* docker container is not stopping
* Stop docker container in test
* closes#658. Also removed wrapper new Error object in all catch blocks in Storage.ts file.
* Added validation in getManagementCredentials function
* Removed es6-promise
* Adding deployment obboarding functionality
* Added .env for credentials
* Updated options and connected with init
* Updated options and connected with init
* addedd unit tests with mocks
* fixed tslinit error by renaming resolve
* Updated with feedback
* throwing error when storage ccount access ket is not available
* Init infra spk command for deploying resources
* Added WIP for init.ts (#15)
* added infra init command (WIP)
* Working terraform init (brute force)
* updated init, with exec & arg parser for templates
* spk infra init with unit testing (#26)
* Adding infra create tests
* modifying spk infra tests
* Updated documentation
* adding check for bedrock source
* Adding additional checks for Bedrock source
* modified spk infra create test
* Removing test infra version (used for initial testing)
* Readme updates for spk infra create
* Removing references to spk infra version decorator
* simplify imports
* Add JSDoc
* Reverting to original working dir and adding jsdoc
* increasing timeout, failed on 10 secs
* update README, increase timeout
* camelcasing vars
* convert let to const
* change const to let for create.ts
* Linting on branch