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.

Sign up for GitHub

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

Jump to bottom

[Master] #1051

Merged
merged 18 commits into from
Mar 13, 2023
Merged

[Master] #1051

merged 18 commits into from
Mar 13, 2023

Conversation

rem1niscence
Copy link
Contributor

merges #1050

Pascal van Leeuwen and others added 18 commits February 23, 2023 11:15
fix: removed unneeded checks for required fields from PHD Client
61a6cb3
@rem1niscence
infra: change phd api key env variable
9a60418
@rem1niscence
lint: fix lint
35c8bef
@rem1niscence
infra: fix env name
1eac759
chore: disable dockerhub action
8d4ee95
@rem1niscence
lint: run linter
4c4098c
@rem1niscence
Merge pull request #1044 from pokt-foundation/change-phd-api-key-env
a556841 
T-18935 infra: change phd api key env variable
@rem1niscence
fix: env name
a1bb1f6
@rem1niscence
Merge pull request #1045 from pokt-foundation/disable-docker-action
366c43c 
chore: disable dockerhub action
@rem1niscence
Merge pull request #1043 from pokt-foundation/phd-client-remove-model…
d1fd6f0 
…-fields

T-18925 fix: removed unneeded checks for required fields from PHD Client
@rem1niscence
lint: fix lint issues
4573ab3
@rem1niscence
fix: temp
6ad0032
@rem1niscence
Merge pull request #1047 from pokt-foundation/fix-env-key-phd
1cee5d9 
fix: env name
@rem1niscence
chore: fix infra deployment files spacing issues
430fc8a
@rem1niscence
Merge pull request #1048 from pokt-foundation/change-euc-canary-config
953601e 
Change eu-central-1 canary config
@TakGN
remove ap-northeast-1 workflow
5bc2470
@rem1niscence
Merge pull request #1049 from pokt-foundation/remove-ap-northeast-1-w…
a53c495 
…orkflow

remove ap-northeast-1 workflow
@rem1niscence
Merge pull request #1050 from pokt-foundation/develop
17d5313 
[staging]
@rem1niscence rem1niscence requested a review from commoddity March 2, 2023 22:51
@rem1niscence rem1niscence requested a review from a team as a code owner March 2, 2023 22:51
@height
Copy link

height bot commented Mar 2, 2023

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

commoddity
commoddity approved these changes Mar 2, 2023
View reviewed changes
Copy link
Contributor

@commoddity commoddity left a comment

Choose a reason for hiding this comment

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

LGTM, changes were already reviewed and approved for staging.

@rem1niscence rem1niscence requested a review from crisog March 13, 2023 15:08
@rem1niscence rem1niscence merged commit 48bbd11 into master Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@commoddity commoddity commoddity approved these changes

@crisog crisog crisog approved these changes

@adshmh adshmh adshmh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rem1niscence @adshmh @crisog @commoddity @TakGN