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

Sourcemaps emulator #230

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ciriousjoker
Copy link

Comment on lines 9 to 15
import { names } from '@nx/devkit'
import { readJson, runNxCommandAsync } from '@nx/plugin/testing'

const NPM_SCOPE = '@proj'

export interface ProjectData {
name: string
Copy link
Owner

Choose a reason for hiding this comment

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

@ciriousjoker the e2e CI isn't passing atm. I don't think your fork is in sync with the latest, since this module is now just a barrel export.

Everything else apart from this looks good, so the simplest fix would be to revert modifications in just this file for this PR?

Copy link
Author

Choose a reason for hiding this comment

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

@simondotm Sorry for taking so long, I've moved the changes I made to the old file to the new location. I just added --inspect-functions there

@ciriousjoker ciriousjoker force-pushed the sourcemaps-emulator branch from 7a37474 to cd1e51a Compare May 8, 2024 11:59
@ciriousjoker ciriousjoker requested a review from simondotm May 8, 2024 12:00
@ciriousjoker
Copy link
Author

@simondotm any news?

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

Successfully merging this pull request may close these issues.

2 participants