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

Typescript 5 #418

Closed
wants to merge 5 commits into from
Closed

Typescript 5 #418

wants to merge 5 commits into from

Conversation

Peeja
Copy link
Member

@Peeja Peeja commented Aug 26, 2024

Enables @import JSDoc tag for #417. Upgrading TypeScript tends to be pretty safe, and if tsc passes, it should be good.

Also adds a packageManager field to package.json so people like me don't get confused over which repos use npm and which use pnpm. 😄

@Peeja Peeja mentioned this pull request Aug 29, 2024
2 tasks
@hannahhoward hannahhoward changed the base branch from main to fix/disable-filecoin-test-for-now August 30, 2024 03:11
@hannahhoward hannahhoward changed the base branch from fix/disable-filecoin-test-for-now to main August 30, 2024 03:12
@hannahhoward hannahhoward changed the base branch from main to fix/disable-filecoin-test-for-now August 30, 2024 03:16
currently, seed.run fails 90%+ of the time because of the Filecoin test.

until we have a more reliable way to test the filecoin flow, the PR disables it.

the inserted comment suggests a few solutions -- either spinning up
custom infra for w3filecoin pipeline on each pr to get more reliable
results, or mocking out the pipeline
@hannahhoward hannahhoward force-pushed the fix/disable-filecoin-test-for-now branch from 346fd91 to 13462fa Compare August 31, 2024 01:25
This reverts commit ba8400a.
@seed-deploy seed-deploy bot temporarily deployed to pr418 August 31, 2024 02:50 Inactive
Base automatically changed from fix/disable-filecoin-test-for-now to main September 19, 2024 19:17
@travis
Copy link
Member

travis commented Oct 9, 2024

@Peeja I'd like to close this out to reclaim IAM resources, wdyt?

@Peeja
Copy link
Member Author

Peeja commented Oct 9, 2024

Looks like the intended changes from this PR have already landed separately. 🚀

@Peeja Peeja closed this Oct 9, 2024
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.

3 participants