Skip to content

Conversation

@bbernays
Copy link
Contributor

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 βœ…

Co-authored-by: Herman Schaaf <hermanschaaf@gmail.com>
@github-actions
Copy link

github-actions bot commented Sep 22, 2023

⏱️ Benchmark results

Comparing with 41c15cf

  • Glob-8 ns/op: 99.33 ⬇️ 0.61% decrease vs. 41c15cf

@codecov
Copy link

codecov bot commented Sep 22, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.03% πŸŽ‰

Comparison is base (1f0c5df) 48.80% compared to head (18caf61) 48.84%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1243      +/-   ##
==========================================
+ Coverage   48.80%   48.84%   +0.03%     
==========================================
  Files          89       89              
  Lines        8310     8310              
==========================================
+ Hits         4056     4059       +3     
+ Misses       3877     3875       -2     
+ Partials      377      376       -1     
Files Changed Coverage Ξ”
scheduler/scheduler.go 55.69% <100.00%> (+1.21%) ⬆️

β˜” View full report in Codecov by Sentry.
πŸ“’ Have feedback on the report? Share it here.

@kodiakhq kodiakhq bot merged commit e53d952 into cloudquery:main Sep 22, 2023
hermanschaaf pushed a commit that referenced this pull request Sep 25, 2023
πŸ€– I have created a release *beep* *boop*
---


##
[4.11.0](v4.10.2...v4.11.0)
(2023-09-25)


### Features

* Provide User with actionable error message when no tables are
configured for syncing
([#1243](#1243))
([e53d952](e53d952))


### Bug Fixes

* **deps:** Update github.com/cloudquery/arrow/go/v14 digest to 64e27ff
([#1245](#1245))
([ff074f4](ff074f4))
* Set GOOS and GOARCH in package command
([#1246](#1246))
([119f962](119f962))

---
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.

bug: Lack of clear error when no tables selected in config

2 participants