Skip to content

feat: Provide User with actionable error message when no tables are configured for syncing#1243

Merged
kodiakhq[bot] merged 7 commits intocloudquery:mainfrom
bbernays:no-table-error
Sep 22, 2023
Merged

feat: Provide User with actionable error message when no tables are configured for syncing#1243
kodiakhq[bot] merged 7 commits intocloudquery:mainfrom
bbernays:no-table-error

Conversation

@bbernays
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown

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