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

v3 colorLogs #523

Closed
wants to merge 159 commits into from
Closed

v3 colorLogs #523

wants to merge 159 commits into from

Conversation

cleve-fauna
Copy link
Contributor

@cleve-fauna cleve-fauna commented Dec 13, 2024

Ticket(s): FE-###

Problem

Explain the context and why you're making that change. What is the problem you're trying to solve? In some cases there is not a problem and this can be thought of being the motivation for your change.

Solution

Describe the modifications you've done.

Result

What will change as a result of your pull request? Note that sometimes this section is unnecessary because it is self-explanatory based on the solution.

Testing

Describe the manual and automated tests you completed to verify the change is working as expected.

echo-bravo-yahoo and others added 30 commits October 3, 2024 17:21
* poc for new login auth

* remove unused code from poc

* no need client secret for auth code

* some fixups

* better port selection

* avoid race condition

* Update src/commands/login.ts

Co-authored-by: Cleve Stuart <[email protected]>

* Update src/commands/login.ts

Co-authored-by: Cleve Stuart <[email protected]>

* surface error_description and leverage open() to for oauth prompt

* move login command to yargs

---------

Co-authored-by: Cleve Stuart <[email protected]>
Co-authored-by: Ashton Eby <[email protected]>
* add schema push command

* add schema pull, status commands

* address pr feedback
* improve schema commands, add fetch wrapper, add verbosity controls

* breakout frontdoor client and get login working

* improve error handling, logging, standardize help text capitalization

* wip

* wip

* wip

* finish error handling tests, behavior

* more fixes

* basic login test with mocked oauth and account

---------

Co-authored-by: Ashton Eby <[email protected]>
this commit:
  - adds several tests for schema commands
  - adds mocha root hooks to clean up sinon
  - adds sinon-chai for better assertion syntax
  - swaps @cloudcmd/stub for sinon in most places
  - rewrites schema pull helpers to parallelize work
also fixes a few bugs in those commands and stubs out the functionality
to test for other commands.
* extend makeFaunaRequest to include request body

* rewrite schema push to use makeFaunaRequest

* fix URL constructors and error handlers

* remove @cloudcmd/stub in favor of sinon

* add test watch mode and junit reporter

* Updated config.yml
* watch mode doesn't work; change script back to one-shot

* normalize paths and resolve ~ in paths

* clean up types, enable @ts-check in more places

* fix schema push and pull commands

also adds a few types and another test
* delete unused files

* remove unused dependencies and package.json scripts

* upgrade eslint, remove babel

* update awilix and open

* upgrade prettier

* upgrade types

* upgrade inquirer

* upgrade husky

* clean up entrypoint
cleve-fauna and others added 29 commits December 9, 2024 10:31
* prettify auth page

* appease liner

* just make it a string

* fix
* Rename "extra" option to "raw"

* formatting

---------

Co-authored-by: E. Cooper <[email protected]>
* don't try to use js completions

* fix v4 completions

* fix v4 completions
* Update output to support fql
* Move to sync and support fql directly via tm grammar

---------

Co-authored-by: Paul Paterson <[email protected]>
* handle accountKey arg with user and secret

* Update src/lib/command-helpers.mjs

Co-authored-by: James Rodewig <[email protected]>

* use camel case

---------

Co-authored-by: James Rodewig <[email protected]>
* Update use of schema API for pull command
* Add support for rendering multiple performance hints and the summary in general
* handle no local schema on

* appease linter

* address comments

* fix tests

* Update test/schema/status.mjs

Co-authored-by: James Rodewig <[email protected]>

---------

Co-authored-by: James Rodewig <[email protected]>
* warn user if no fsl files are found

* remove .only

* throw a validation error

* fix import

---------

Co-authored-by: E. Cooper <[email protected]>
* RFC local conatiner commands

* Log progress

* Use stderrStream

* Remove -q

* No jumps when writing output

* Fix build

* Improvements for states

* Marke awaits in loop OK

* Refactor

* Refactor

* Refactor

* Initial tests

* Initial tests

* Initial tests

* Fix stuff

* More tests

* More tests

* Move test file to be for local command. Remove ability to pick the image

* Apply suggestions from code review

Co-authored-by: echo-bravo-yahoo <[email protected]>

---------

Co-authored-by: echo-bravo-yahoo <[email protected]>
* don't use a default profile

* add test
* Redact secrets in logArgv

* Display first and last 4 if the secret meets a minimum length
…ntrol hostIp. (#512)

* Tests for health checks.

* Argument checks

* Skip pull test

* Tests for pull failing

* Tests for pull failing

* Support for hostIp

* Port occupied tests

* Remove only

* Fail gracefully if the container is already present but on a different port.

* Doc fixes
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.

7 participants