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

Delete hermes-urls.d.ts #280

Merged
merged 1 commit into from
Aug 8, 2023
Merged

Conversation

jeffdaley
Copy link
Contributor

@jeffdaley jeffdaley commented Aug 8, 2023

Removes problematic d.ts file causing build problems on staging.

export const HERMES_GITHUB_REPO_URL: string;
export const TEST_SUPPORT_URL: string;
export const TEST_SHORT_LINK_BASE_URL: string;
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I shouldn't have been trying to export consts from a d.ts in the first place.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I made this to appease Mirage but it turns out not to need it.

@jeffdaley jeffdaley marked this pull request as ready for review August 8, 2023 15:34
@jeffdaley jeffdaley requested a review from a team as a code owner August 8, 2023 15:34
@jeffdaley jeffdaley merged commit 63aed11 into main Aug 8, 2023
2 checks passed
@jeffdaley jeffdaley deleted the jeffdaley/typescript-definition-files branch August 8, 2023 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants