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

Todomvc-vue: source-map creation #386

Merged
merged 2 commits into from
Mar 2, 2024

Conversation

flashdesignory
Copy link
Contributor

@flashdesignory flashdesignory commented Mar 1, 2024

This might fix #385 (comment)

Open for better solutions...

@kara

README.md Outdated Show resolved Hide resolved
@lpardosixtosMs
Copy link
Contributor

Could you rebuild the complex version too?

@rniwa rniwa added the trivial change A change that doesn't affect benchmark results label Mar 1, 2024
@rniwa
Copy link
Member

rniwa commented Mar 1, 2024

There is no test content change here, right?

@flashdesignory
Copy link
Contributor Author

There is no test content change here, right?

correct - I made a config change for the build process and run the build command.

@flashdesignory
Copy link
Contributor Author

just rebased with main, to make the readme file change go away...

Copy link
Contributor

@bgrins bgrins left a comment

Choose a reason for hiding this comment

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

It's very hard for me to understand the PR diff, but I trust that this doesn't affect runtime so feel free to land it (cc @julienw)

@flashdesignory flashdesignory merged commit 7dc43d2 into WebKit:main Mar 2, 2024
4 checks passed
@flashdesignory flashdesignory deleted the todomvc-vue/src-maps branch March 2, 2024 00:20
flashdesignory added a commit that referenced this pull request Mar 2, 2024
flashdesignory added a commit that referenced this pull request Mar 2, 2024
@flashdesignory flashdesignory restored the todomvc-vue/src-maps branch March 2, 2024 01:19
@flashdesignory flashdesignory deleted the todomvc-vue/src-maps branch May 20, 2024 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
trivial change A change that doesn't affect benchmark results v3-blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Vue TodoMVC source maps include full path of author's machine
4 participants