Skip to content

Fix most module cyclic references #4978

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

Merged
merged 1 commit into from
Nov 10, 2017

Conversation

hshristov
Copy link
Contributor

There is one left for ios and android:

  • http -> http-request -> image-source
    that can't be fixed easily and will be removed once we delete image-source module because it is obsolete anyway.

And one more for android only:

  • frame -> fragment
    that could be removed if we use global object.

Updated madge version

…droid:

- http -> http-request -> image-source
that can't be fixed easily and will be removed once we delete image-source module because it is obsolete anyway.
There is one more for android:
- frame -> fragment that could be removed if we use global object.
Updated madge bersion
@ghost ghost assigned hshristov Oct 24, 2017
@ghost ghost added the in progress label Oct 24, 2017
@hshristov
Copy link
Contributor Author

uitests-ios-pickers

export * from "../core/view";

function onLivesync(args: EventData): void {
// give time to allow fileNameResolver & css to reload.
Copy link
Contributor

Choose a reason for hiding this comment

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

Mind the setTimeout

@SvetoslavTsenov
Copy link
Contributor

ios-issues

@hshristov hshristov merged commit 43fbabb into master Nov 10, 2017
@hshristov hshristov deleted the hhristov/fix-module-cyclic-references branch November 10, 2017 13:47
@lock
Copy link

lock bot commented Aug 26, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Aug 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants