Skip to content

fix: Remove empty test and enable some lint rules#90

Merged
erezrokah merged 2 commits intocloudquery:mainfrom
erezrokah:fix/lint
Sep 13, 2022
Merged

fix: Remove empty test and enable some lint rules#90
erezrokah merged 2 commits intocloudquery:mainfrom
erezrokah:fix/lint

Conversation

@erezrokah
Copy link
Copy Markdown
Member

Summary

unused-parameter is reported on the deleted file


Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines 🧑‍🎓
  • Run go fmt to format your code 🖊
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests 🧪
  • Ensure the status checks below are successful ✅

@erezrokah erezrokah merged commit 514fba4 into cloudquery:main Sep 13, 2022
@erezrokah erezrokah deleted the fix/lint branch September 13, 2022 06:55
kodiakhq bot pushed a commit that referenced this pull request Sep 13, 2022
🤖 I have created a release *beep* *boop*
---


## [0.5.0](v0.4.2...v0.5.0) (2022-09-13)


### ⚠ BREAKING CHANGES

* Enable var names lint rule and fix issues (#88)
* Disable default completion command (#96)

### Features

* Disable default completion command ([#96](#96)) ([67fca4b](67fca4b))


### Bug Fixes

* Enable var names lint rule and fix issues ([#88](#88)) ([4a752b5](4a752b5))
* Remove description from docs ([#92](#92)) ([7df58df](7df58df))
* Remove empty test and enable some lint rules ([#90](#90)) ([514fba4](514fba4))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants