Skip to content

Bnr/copy wrapper #1122

New issue

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

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

Already on GitHub? # to your account

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
…

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants