Skip to content

Provide better error messages#479

Merged
nasc17 merged 1 commit intomaster_GA_2.0.0from
nasc/ga/fixErrorMessaging
Sep 25, 2023
Merged

Provide better error messages#479
nasc17 merged 1 commit intomaster_GA_2.0.0from
nasc/ga/fixErrorMessaging

Conversation

@nasc17
Copy link
Contributor

@nasc17 nasc17 commented Sep 22, 2023

@nasc17 nasc17 requested a review from a team as a code owner September 22, 2023 21:20
DaeunYim
DaeunYim approved these changes Sep 23, 2023
View reviewed changes
Copy link
Contributor

@DaeunYim DaeunYim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Really good catch!

@nasc17 nasc17 merged commit 674abd2 into master_GA_2.0.0 Sep 25, 2023
nasc17 added a commit that referenced this pull request Dec 18, 2023
@nasc17
Add full error with more detail (#479) (#486)
d15e576 
* add full error with more detail (#479)

* Preparing version 1.10.0 Snap (#478) (#487)

* update to OrcaSQL ESRP Code Sign

* code sign for non-windows

* remove pr trigger for snap

* remove trigger

* update date esrp key for linux
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@samir-puranik samir-puranik Awaiting requested review from samir-puranik

1 more reviewer

@DaeunYim DaeunYim DaeunYim approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@nasc17 @DaeunYim