Skip to content
This repository has been archived by the owner on Jul 28, 2021. It is now read-only.

Ecmascript20 #32

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Ecmascript20 #32

wants to merge 3 commits into from

Conversation

trullock
Copy link

Fixes #29 by updating EcmaScript.NET reference

Had to update to Net4.5

I've had to drop CLS compliance because EcmaScript.NET now isnt CLS compliant under Net4.5.

I'd like to see this not be a problem for a 3.0.0.0 (maybe pre-release) version. I'll help get a new version of EcmaScript.NET CLS compliant in the background, but for now I'd really benefit from a YUIC that doesnt have this bug

@trullock
Copy link
Author

I see the other PRs have .net version upgrades and pushback on wanting netstandard support.

I'm happy to help do this

Shall we discuss that on #22 and decide what we want?

@freeranger
Copy link
Collaborator

@trullock Why was the net45 upgrade necessary for this PR?

@trullock
Copy link
Author

because Ecmascript.Net 2.0 which contains the bugfix has been updated to 45

@PureKrome
Copy link
Member

@trullock more info now in - please refer to #22 . Can this PR either be closed and a new one opened OR this one updated?

@PureKrome
Copy link
Member

With PR #33 merged, can this be now closed @trullock ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants