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

Publish new version (ie. from 0.9.25 to 0.9.26) #191

Closed
silbinarywolf opened this issue Feb 1, 2019 · 20 comments
Closed

Publish new version (ie. from 0.9.25 to 0.9.26) #191

silbinarywolf opened this issue Feb 1, 2019 · 20 comments

Comments

@silbinarywolf
Copy link
Contributor

silbinarywolf commented Feb 1, 2019

As per this PR comment:
#189 (comment)

Waiting on @MeirionHughes to reply back and give @AureliaEffect npm publishing rights.
cc @EisenbergEffect

I wanted to create an explicit issue that's open so other users can be made aware of what we're waiting on.

@MeirionHughes
Copy link
Contributor

MeirionHughes commented Feb 4, 2019

Just noticed this; I've added aureliaeffect as owner to all the packages.

aurelia-template-lint
gulp-aurelia-template-lint
template-lint
gulp-template-lint

@silbinarywolf
Copy link
Contributor Author

Thanks! I appreciate you taking the time to do this in your own time!

@silbinarywolf
Copy link
Contributor Author

silbinarywolf commented Feb 13, 2019

@EisenbergEffect Are we able to get this published soon? Just an ETA so I know when to expect this to be updated will make me happy :)

@EisenbergEffect
Copy link
Contributor

Just got internet back after snow storm took it out for a day and a half :) I'm looking into this now. Might take a day or two since I need to look into details a bit more and see if there's anything special in the publish process. Thanks for your patience @silbinarywolf

@silbinarywolf
Copy link
Contributor Author

Ah thats rough! We've had flooding / heavy rain down here too lately!
There's no urgency on this so don't feel the need to rush to it, I'd just like to have a lingering todo item covered off within the next month or so!

@EisenbergEffect
Copy link
Contributor

Sadly, this repo doesn't build 😢I'm going to have to dig in and try to find out what's going on. Might take a bit of time.

@EisenbergEffect
Copy link
Contributor

EisenbergEffect commented Mar 5, 2019

I've pushed a new version to master that updates some dependencies. The current issue appears to be that the TypeScript compiler APIs have changed. Since I'm less familiar with the internal AST APIs, I'm not sure what the correct way is to convert from the old code to the new code. The errors are in one file: source/rules/bindings.ts. Does anyone here have familiarity with this who could submit a PR? Also including @fkleuver since I know he's messed with TS compiler APIs a bit as well and may be able to make a quick fix. It's only six errors, very similar in nature, that need correction. I can dig further myself, but thought I would put it out to the group here to see if anyone already had this knowledge first.

@silbinarywolf
Copy link
Contributor Author

That's essentially the issue I hit while trying to update this. The AST APIs have all changed.

@EisenbergEffect
Copy link
Contributor

I'm hoping the fix is obvious to @fkleuver :) If not or if he doesn't get around to it, then I'll plan to tackle this in a few days.

@fkleuver
Copy link
Member

It appeared to be a fairly straight-forward fix (that is, if things actually still work). See #192
@silbinarywolf @EisenbergEffect

I can't get the rests to run:
image

@EisenbergEffect
Copy link
Contributor

Thanks for taking time to look into this @fkleuver I was going to sit down today and figure it out...so your timing is impeccable 🤣

@silbinarywolf Can you try this out in your app and see if it gets you going again?

@silbinarywolf
Copy link
Contributor Author

Not sure what you mean by "get us going again"?

The current tagged version is functioning OK, it'll just be painful if somebody on my team hits the bug I fixed in my PR from a couple of months ago :) ie. unclosed HTML tags.

@EisenbergEffect
Copy link
Contributor

I mainly meant, can you confirm if what's in master now works for you, now that we've updated the compiler and got it building again.

@silbinarywolf
Copy link
Contributor Author

Yep sure. I'll set aside some time this week to do this :)

@EisenbergEffect
Copy link
Contributor

Awesome. Thank you @silbinarywolf !

@silbinarywolf
Copy link
Contributor Author

silbinarywolf commented Mar 18, 2019

Its working fine for my use-case! However it's worth noting that we currently don't have typeChecking running over the project. (We've yet to fix up all the errors its throwing as we've installed the plugin ~3 years after the project was started)

EDIT: cc @EisenbergEffect

@EisenbergEffect
Copy link
Contributor

Excellent. Thanks!

@EisenbergEffect
Copy link
Contributor

I published 0.10.0.

@silbinarywolf
Copy link
Contributor Author

@EisenbergEffect Thanks! Perhaps tag the Git repo with that version too :)

@EisenbergEffect
Copy link
Contributor

Yep, that's next :)

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

No branches or pull requests

4 participants