Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LogpushJob.Filter is optional #1712

Conversation

sbfaulkner
Copy link
Contributor

@sbfaulkner sbfaulkner commented Jun 20, 2022

Note depends on the new release of cloudflare-go

this PR changes the usage of LogpushJob.Filter after the fix in cloudflare/cloudflare-go#937

  • set pointer instead of copying struct to create LogpushJob from resource
  • check nil pointer when creating resource from LogpushJob

@sbfaulkner sbfaulkner force-pushed the optional-filter branch 3 times, most recently from d14cfc9 to 6f1778d Compare June 20, 2022 14:39
@github-actions
Copy link
Contributor

github-actions bot commented Jun 20, 2022

changelog detected ✅

@github-actions
Copy link
Contributor

This pull request contains a CHANGELOG.md file which should only be modified by maintainers.

If you are looking to include a CHANGELOG entry, you should use the process documented at https://github.com/cloudflare/terraform-provider-cloudflare/blob/master/contributing/changelog-process.md instead.

In order for this pull request to be merged, you need remove the modifications to CHANGELOG.md.

@sbfaulkner
Copy link
Contributor Author

rebased to not include changes from our fork

@github-actions
Copy link
Contributor

changelog detected ✅

1 similar comment
@github-actions
Copy link
Contributor

changelog detected ✅

@jacobbednarz jacobbednarz changed the base branch from master to dependabot/go_modules/github.com/cloudflare/cloudflare-go-0.42.0 June 22, 2022 04:55
@jacobbednarz jacobbednarz merged commit ba9c132 into cloudflare:dependabot/go_modules/github.com/cloudflare/cloudflare-go-0.42.0 Jun 22, 2022
@sbfaulkner sbfaulkner deleted the optional-filter branch June 22, 2022 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants