Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

fix: Sysctl freebsd #370

Merged
merged 4 commits into from
Jun 26, 2022
Merged

fix: Sysctl freebsd #370

merged 4 commits into from
Jun 26, 2022

Conversation

roneli
Copy link
Contributor

@roneli roneli commented Jun 23, 2022 •
edited

Summary

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 ✅

@roneli roneli requested a review from a team as a code owner June 23, 2022 14:46
@roneli roneli requested review from bbernays and removed request for a team June 23, 2022 14:46
roneli and others added 2 commits June 23, 2022 18:09
@roneli @disq
Update helpers/limit/sysctl_freebsd.go
8f41338 
Co-authored-by: Kemal <223029+disq@users.noreply.github.com>
@roneli
Update sysctl_windows.go
77ce6ae
@roneli roneli merged commit f52efe9 into cloudquery:main Jun 26, 2022
@roneli roneli deleted the fix-sysctl-freebsd branch June 26, 2022 10:23
@cq-bot cq-bot mentioned this pull request Jun 26, 2022
Merged
@roneli roneli mentioned this pull request Jun 26, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@disq disq disq left review comments

@erezrokah erezrokah erezrokah approved these changes

@bbernays bbernays Awaiting requested review from bbernays bbernays was automatically assigned from cloudquery/cloudquery-opensource

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@roneli @disq @erezrokah