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

[dev] main.js built from Linux different than one built from Windows #220

Open
zijchen opened this issue May 13, 2024 · 0 comments
Open
Labels
backlog Planned for future bug Something isn't working dependencies Pull requests that update a dependency file

Comments

@zijchen
Copy link
Contributor

zijchen commented May 13, 2024

The final output is the same size but the order of the exports has one difference. No breaking functionality as the order doesn't matter during runtime, but breaks our check-lib workflow.

This is the block that is different:

2609: function(e, t, n) {
    "use strict";
    var i = this && this.__awaiter || function(e, t, n, i) {
        return new(n || (n = Promise))((function(r, o) {
            function s(e) {
                try {
                    c(i.next(e))
                } catch (e) {
                    o(e)
                }
            }

            function a(e) {
                try {
                    c(i.throw(e))
                } catch (e) {
                    o(e)
                }
            }

            function c(e) {
                var t;
                e.done ? r(e.value) : (t = e.value, t instanceof n ? t : new n((function(e) {
                    e(t)
                }))).then(s, a)
            }
            c((i = i.apply(e, t || [])).next())
        }))
    };
    Object.defineProperty(t, "__esModule", {
        value: !0
    });
    const r = n(2225),
        o = n(4458);
    t.AuthorizerFactory = class {
        static getAuthorizer() {
            return i(this, void 0, void 0, (function*() {
                r.debug("try-get AzureCLIAuthorizer");
                try {
                    return yield o.AzureCLIAuthorizer.getAuthorizer()
                } catch (e) {
                    throw r.debug(e), new Error("No credentails found. Add an Azure login script before this action.")
                }
            }))
        }
    }
},
@zijchen zijchen added the need-to-triage Requires investigation label May 13, 2024
@zijchen zijchen added bug Something isn't working dependencies Pull requests that update a dependency file backlog Planned for future and removed need-to-triage Requires investigation labels May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Planned for future bug Something isn't working dependencies Pull requests that update a dependency file
Projects
None yet
Development

No branches or pull requests

1 participant