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

Bnr/copy wrapper #1122

Draft
wants to merge 33 commits into
base: main
Choose a base branch
from
Draft

Bnr/copy wrapper #1122

wants to merge 33 commits into from

Conversation

bryans99
Copy link
Collaborator

👋👋 Thank you for contributing to Looker sdk-codegen (⚡️🍣)

📺 Watch a quick video on how to you make your pull requests fast and easy for you and your reviewer(s):
https://threadit.app/thread/k45nyrqugoinw6xub9vf/message/fl8m57exhqg4r0a4hg01eebn

  • 👆 Make sure your pull request title follows Pull Request Title Guidelines from our Contribution guide
  • 👉 Don't forget to replace these instructions with your ✨awesome✨ description of what this change actually does. Additionally, it's great to include context on how it works and why the change was needed.
  • 👇 Edit "Developer Checklist" to reflect items relevant to this PR (and try to make sure to check everything off before asking for review)

Developer Checklist ℹ️

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

patnir41 and others added 30 commits July 14, 2022 01:19
@bryans99 bryans99 marked this pull request as draft July 21, 2022 16:52
@github-actions
Copy link
Contributor

APIX Tests

0 files   -     1  0 suites   - 80   0s ⏱️ - 5m 39s
0 tests  - 328  0 ✔️  - 315  0 💤  - 13  0 ❌ ±0 
0 runs   - 344  0 ✔️  - 331  0 💤  - 13  0 ❌ ±0 

Results for commit e806099. ± Comparison against base commit 0eb10ee.

This pull request removes 328 tests.
 DocTitle renders a heading with title prop content ‑  DocTitle renders a heading with title prop content
 MethodBadge renders with the provided verb ‑  MethodBadge renders with the provided verb
ApiSpecSelector it lists all available specs ‑ ApiSpecSelector it lists all available specs
ApiSpecSelector requests selected spec ‑ ApiSpecSelector requests selected spec
ApiSpecSelector the base spec is selected by default ‑ ApiSpecSelector the base spec is selected by default
Banner renders with button that opens api 4.0 dev portal link in new page ‑ Banner renders with button that opens api 4.0 dev portal link in new page
Banner sets local storage value and unrenders on close button click ‑ Banner sets local storage value and unrenders on close button click
BrowserAdaptor returns correct font overrides ‑ BrowserAdaptor returns correct font overrides
CodeCopy displays code and clipboard UI ‑ CodeCopy displays code and clipboard UI
CodeDisplay it highlights text matching search pattern ‑ CodeDisplay it highlights text matching search pattern
…

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.

3 participants