-
Notifications
You must be signed in to change notification settings - Fork 49
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
feat: install TypeScript #148
Conversation
@minutebase @rlivsey Any interest in this? |
I've started adding type definitions for It would be great to have types for this lib indeed 🤗 ping @Exelord |
I know, need that as well :) Ember-can is awaiting a major upgrade soon. |
is this upgrade already started, or is it about to? And in parallel, do you want more PRs with type definitions like this one? |
It's happening :) some changes just landed. TS will be welcome for upcoming v5 |
@Exelord Is it still happening? 😋 |
Unfortunately I don't have enough time to work on it anymore :/ so the progres on next upgrade is on hold. |
@Exelord Is there anything we could help with to either merge this PR or move the V5 forward? 🙏 |
Hey @esbanarango, As mentioned, currently I do not have time to maintain this package. I am really sorry for that. I would recommend maintaining own fork with changes that are needed for your project taking into account how small this package is. |
@Exelord are you able to hand over maintenance to others who do have the time? Completely understood that you have other time constraints on your end, yet there's still interest in having a shared add-on that the community uses for this ... |
@rlivsey would be nice to transfer this addon to adopted-ember-addons or to get a new maintainer, so that the addon can be migrate to TS and addon v2 |
I cannot move the repo, but as the author is not responding, I will grant maintenance permission. Who would like to take it over? |
@Exelord I can help |
I'm also willing to help. Getting this in would be great 😄 |
Done. Please sync with each other on next steps :) @RobbieTheWagner @esbanarango |
Thanks @Exelord! @esbanarango do you have the cycles to take a look at getting this ready to merge? I won't have availability for probably a few weeks. |
# Conflicts: # addon/services/can.ts # package-lock.json # package.json
@RobbieTheWagner Yeah, I'll take a look! @charlesfries Let me know once this is ready 🤝 |
@esbanarango should be good now |
@@ -87,7 +119,9 @@ | |||
"prettier": "^2.5.0", | |||
"qunit": "^2.17.2", | |||
"qunit-dom": "^2.0.0", | |||
"sinon": "^12.0.1", | |||
"rimraf": "^5.0.5", | |||
"sinon": "^17.0.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needed to update this to fix a build error:
ERROR in ./node_modules/@sinonjs/fake-timers/src/fake-timers-src.js 104:479-504
Module not found: Error: Can't resolve 'util' in 'C:\Users\xxx\ember-can\node_modules\@sinonjs\fake-timers\src'
@@ -54,11 +57,40 @@ | |||
}, | |||
"devDependencies": { | |||
"@ember/optional-features": "^2.0.0", | |||
"@ember/test-helpers": "^2.6.0", | |||
"@ember/test-helpers": "^3.3.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needed to update this to fix a Glint TS error:
node_modules/@glint/environment-ember-loose/-private/dsl/integration-declarations.d.ts:107:16 - error TS2665: Invalid module name in augmentation. Module '@ember/test-helpers' resolves to an untyped module at 'C:/Users/xxx/ember-can/node_modules/@ember/test-helpers/index.js', which cannot be augmented.
# Conflicts: # package-lock.json # package.json
@types/ember-data__model
AbilitiesService