Skip to content

Conversation

@sadevelopment
Copy link

This has package renaming, replacement of TeamHubApp with GitLiveApp. NPM publishing to main registry

Copy link
Member

@nbransby nbransby left a comment

Choose a reason for hiding this comment

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

are you going to delete the pom?

.gitignore Outdated
.gradle/

/node_modules
/src/commonMain/kotlin/.DS_Store
Copy link
Member

Choose a reason for hiding this comment

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

update to all .DS_Stores

build.gradle.kts Outdated
}
dependsOn(copyPackageJson, copyJS, copySourceMap)
workingDir("$buildDir/node_module")
if(Os.isFamily(Os.FAMILY_WINDOWS)) {
Copy link
Member

Choose a reason for hiding this comment

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

can remove this if now

Copy link
Author

Choose a reason for hiding this comment

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

Do you mean just Os.isFamily?

Copy link
Author

Choose a reason for hiding this comment

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

Sorry misread it

Copy link
Member

Choose a reason for hiding this comment

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

yeah the only reason to make it OS specific was to run 2 command in one line which youi have removed now

@nbransby nbransby merged commit 23920b1 into master Mar 25, 2020
@nbransby nbransby deleted the publish_to_central branch March 25, 2020 15:46
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.

3 participants