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

Add types to core #403

Merged
merged 12 commits into from
Mar 29, 2023
Merged

Add types to core #403

merged 12 commits into from
Mar 29, 2023

Conversation

Grubba27
Copy link
Contributor

This relates to meteor/meteor#12428
and also closes #388
I could not test it in the test app but if we publish it we maybe can in a newly created app?

distalx and others added 11 commits July 24, 2022 09:29
Bumps [moment-timezone](https://github.com/moment/moment-timezone) from 0.5.34 to 0.5.37.
- [Release notes](https://github.com/moment/moment-timezone/releases)
- [Changelog](https://github.com/moment/moment-timezone/blob/develop/changelog.md)
- [Commits](moment/moment-timezone@0.5.34...0.5.37)

---
updated-dependencies:
- dependency-name: moment-timezone
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
…nt-timezone-0.5.37

Bump moment-timezone from 0.5.34 to 0.5.37 in /site
Bumps [decode-uri-component](https://github.com/SamVerschueren/decode-uri-component) from 0.2.0 to 0.2.2.
- [Release notes](https://github.com/SamVerschueren/decode-uri-component/releases)
- [Commits](SamVerschueren/decode-uri-component@v0.2.0...v0.2.2)

---
updated-dependencies:
- dependency-name: decode-uri-component
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
…de-uri-component-0.2.2

Bump decode-uri-component from 0.2.0 to 0.2.2 in /site
@jankapunkt jankapunkt changed the base branch from master to release-2.6.2 January 11, 2023 14:41
@jankapunkt jankapunkt added this to the Blaze 2.6.2 milestone Jan 11, 2023
@jankapunkt
Copy link
Collaborator

I changed the target to release-2.6.2 I think we should publish this asap, right?

@Grubba27
Copy link
Contributor Author

Yes! give it a try. Create a new app and test if typedefs are in this folder.

  • Create an app.
  • Add zodern:types
  • tsconfig.json
  • run meteor lint
  • check if was created the files and folders below and if there are a link to the blaze type definitions
    Screenshot 2023-01-11 at 11 44 15

@zodern
Copy link
Collaborator

zodern commented Jan 11, 2023

It is possible to use types from local packages, so you can try it before publishing.

@Grubba27
Copy link
Contributor Author

I tried but I was not successful trying locally, the steps that I've made are:

  • added zodern types to project
  • tsconfig.json
  • ran meteor lint

the packages.d.ts stood blank :/

@zodern
Copy link
Collaborator

zodern commented Jan 11, 2023

How did you configure Meteor to find the blaze packages? I usually do it by adding the blaze/packages folder to METEOR_PACKAGE_DIRS.

You can check if Meteor found the local blaze package by running meteor list. It will indicate which packages are local.

@Grubba27
Copy link
Contributor Author

I ran ln -sfn ../packages ./packages

Copy link
Collaborator

@StorytellerCZ StorytellerCZ left a comment

Choose a reason for hiding this comment

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

LGTM

@StorytellerCZ StorytellerCZ requested a review from zodern March 23, 2023 14:38
@StorytellerCZ
Copy link
Collaborator

@jankapunkt @zodern I'm fine with merging this and releasing this asap.

@jankapunkt jankapunkt merged commit c0d40bf into release-2.6.2 Mar 29, 2023
@StorytellerCZ StorytellerCZ deleted the add-types-to-core branch March 31, 2023 06:34
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.

Add ts types
5 participants