Skip to content
This repository was archived by the owner on Jun 16, 2022. It is now read-only.

Conversation

@lseppala
Copy link
Contributor

@lseppala lseppala commented May 14, 2022

This auto-corrects lint identified by eslint and removes the search for go.sum (since go.mod and go.sum are always paired, no need to search twice).

This also adds missing dev dependencies and other config fixes. npm run all now works on a Codespace.

@lseppala lseppala requested review from lorenanicole and mrysav and removed request for lorenanicole May 14, 2022 01:07
@lseppala lseppala changed the title Lint and remove go.sum search Lint, remove go.sum search, and fix 'npm run all' May 14, 2022
Copy link
Contributor

@lorenanicole lorenanicole left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for adding in this!

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah good catch on the license, thanks @lseppala !

@lseppala lseppala merged commit a53b2da into main May 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants