Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

chore: changes for golang ci lint to pass #80

Merged
merged 5 commits into from
Jan 29, 2023

Conversation

rach-id
Copy link
Member

@rach-id rach-id commented Jan 27, 2023

Overview

Mainly comments a function that we will eventually use when we stick the QGB implementation together.

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@rach-id rach-id requested a review from rahulghangas January 27, 2023 09:53
@rach-id rach-id requested a review from evan-forbes as a code owner January 27, 2023 09:53
@rach-id rach-id self-assigned this Jan 27, 2023
@rach-id rach-id changed the title chore: chore add nolint to parameters that will be used later chore: fix golang ci lint Jan 27, 2023
@rach-id rach-id changed the title chore: fix golang ci lint chore: changes for golang ci lint to pass Jan 27, 2023
@rach-id rach-id mentioned this pull request Jan 27, 2023
5 tasks
@codecov-commenter
Copy link

Codecov Report

Merging #80 (c21286d) into main (16eb6c5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #80   +/-   ##
=======================================
  Coverage   17.52%   17.52%           
=======================================
  Files           2        2           
  Lines          97       97           
=======================================
  Hits           17       17           
  Misses         79       79           
  Partials        1        1           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@evan-forbes evan-forbes left a comment

Choose a reason for hiding this comment

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

perhaps we should leave an issue to fix the commented code if not done already, not necessary imho, as I'm sure we'll handle it eventually.

LGTM

@rach-id
Copy link
Member Author

rach-id commented Jan 29, 2023

#81

@rach-id rach-id merged commit 981ff20 into celestiaorg:main Jan 29, 2023
@rach-id rach-id deleted the fix_golang_ci branch January 29, 2023 10:47
rach-id added a commit to rach-id/orchestrator-relayer that referenced this pull request Jan 31, 2023
* chore: chore add nolint to parameters that will be used later

* chore: chore add nolint to parameters that will be used later

* chore: comment unused code for CI to pass

* chore: rename commands for revive to pass
rach-id added a commit that referenced this pull request Jan 31, 2023
* feat: add base DHT implementation and test node

* chore: rename test network to test DHT network

* Update p2p/dht.go

* Update p2p/dht.go

* chore: rename DHTNetwork to TestDHTNetwork

* ci: add markdown lint and run lint on PRs (#67)

* ci: add markdown lint

* ci: update lint to run on pull request

* test: add dcc sign bytes test (#71)

* test: add dcc sign bytes test

* chore: rename byt to buf as suggested by @rahulghangas

* ci: use go version 1.19 in lint (#68)

* chore: changes for golang ci lint to pass (#80)

* chore: chore add nolint to parameters that will be used later

* chore: chore add nolint to parameters that will be used later

* chore: comment unused code for CI to pass

* chore: rename commands for revive to pass

* chore(deps): bump goreleaser/goreleaser-action from 4.1.0 to 4.1.1 (#83)

Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 4.1.0 to 4.1.1.
- [Release notes](https://github.com/goreleaser/goreleaser-action/releases)
- [Commits](goreleaser/goreleaser-action@v4.1.0...v4.1.1)

---
updated-dependencies:
- dependency-name: goreleaser/goreleaser-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ci: update golang ci lint version (#82)

* ci: only run `split-test-files` if go files change (#77)

* feat: add copied relayer implementation (#84)

* feat: add relayer implementation

* chore: refactor relayer to have start and stop methods

* chore(deps): bump actions/cache from 3.2.3 to 3.2.4 (#86)

Bumps [actions/cache](https://github.com/actions/cache) from 3.2.3 to 3.2.4.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@v3.2.3...v3.2.4)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): bump goreleaser/goreleaser-action from 4.1.1 to 4.2.0 (#87)

Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 4.1.1 to 4.2.0.
- [Release notes](https://github.com/goreleaser/goreleaser-action/releases)
- [Commits](goreleaser/goreleaser-action@v4.1.1...v4.2.0)

---
updated-dependencies:
- dependency-name: goreleaser/goreleaser-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: update confirms types and adds marshallers/unmarshallers (#70)

* feat: update confirms types and adds marshallers/unmarshallers

* chore: fix conflicts

* chore: go mod tidy

* chore: increase the DHT sleep to 10ms

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

3 participants