merge #8
Merged
merge #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* First cut at adding enc type factory * Finalized enc type factory * Adding unit test * PyLinting * Adding license * Apply suggestions from code review Co-authored-by: Jesse Yang <jesse.yang@airbnb.com> * Moving things from enc -> encrypt * CI commit * One more fix * Tweaking config name * Fixing broken test * Fixing broken test again Co-authored-by: Jesse Yang <jesse.yang@airbnb.com>
* Create initial helm-release action for superset * Add lintconf to ensure chart passes validation * Add lint-test job * Add apache licence headers * Run job for master only * Move helm-lint to separate workflow * Helm release for master & helm dir changes only * Dummy change to test linter
* chore: type some SQL Lab components * Update superset-frontend/src/SqlLab/components/QueryHistory.tsx Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com> Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>
* fix: Add extra requires for shillelagh * Update google-sheets.mdx * Update index.mdx
* Switching to local codecov script * Adding license / update instructions * Fixing FE build * Fixing FE codecov test
* database errors for mssql * revisions
* chore: rename connection errors * feat: error messages for Presto connections * Add unit tests * Update docs/src/pages/docs/Miscellaneous/issue_codes.mdx Co-authored-by: AAfghahi <48933336+AAfghahi@users.noreply.github.com> Co-authored-by: AAfghahi <48933336+AAfghahi@users.noreply.github.com>
* Add chart-action submodules * use local github actions for chart actions * Run actions checkout for local checkout * Run lint when PR is edited * Remove path filter and dummy value * Fix fetch-depth * Test chart version bump * Update maintainer to pass validation * Update maintainer to pass validation
Codecov Report
@@ Coverage Diff @@
## master #8 +/- ##
==========================================
- Coverage 80.38% 76.05% -4.33%
==========================================
Files 318 945 +627
Lines 25598 47916 +22318
Branches 0 5949 +5949
==========================================
+ Hits 20576 36443 +15867
- Misses 5022 11268 +6246
- Partials 0 205 +205
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
stephenLYZ
pushed a commit
that referenced
this pull request
Dec 1, 2021
* build: migrate to nimbus * fix: eslint-disable statements * fix: conflict * fix: lint * fix: lints * fix: bump packages
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
SUMMARY
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
ADDITIONAL INFORMATION