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

Rename Web to HTML #2997

Merged
merged 5 commits into from
Apr 6, 2023
Merged

Rename Web to HTML #2997

merged 5 commits into from
Apr 6, 2023

Conversation

igordmn
Copy link
Collaborator

@igordmn igordmn commented Apr 6, 2023

  1. Rename "compose.web" to "compose.html" in Gradle DSL. Users will see this:
    image

  2. Rename maven artifacts (with backward compatible "relocation" artifact). In case users don't use Gradle DSL, or when they use a library that depend on the old Compose version.

  3. Rename "web" folder to "html"

An example build: 0.0.0-igor.demin-rename-web-to-html-dev1014

Will do in support/1.4.0 branch:

  1. Move examples/web-* to examples/html/*
  2. Rename Tutorials/Web to Tutorials/HTML
  3. Rename "Compose for Web" to "Compose HTML Library" in the tutorials

igordmn added 3 commits April 6, 2023 11:19
1. Rename "compose.web" to "compose.html" in Gradle DSL
2. Rename maven artifacts (with backward compatible "relocation" artifact)
3. Rename "web" folder to "html"

Will do in support/1.4.0 branch
1. Move examples/web-* to examples/html/*
2. Rename Tutorials/Web to Tutorials/HTML
3. Rename "Compose for Web" to "Compose HTML Library" in the tutorials
@igordmn igordmn requested review from eymar and katia-energizer April 6, 2023 09:55
if "%OS%"=="Windows_NT" endlocal

:omega
@rem
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It seems it was converted to Unix-Style line endings on committing. Bat files works normally on Windows with these line endings.

@eymar
Copy link
Member

eymar commented Apr 6, 2023

In our CI configuration [3. Components. Web. Test], it expects "web" directory. The configuration will need to be changed too.

@igordmn igordmn merged commit 59eda00 into master Apr 6, 2023
@igordmn igordmn deleted the igor.demin/rename-web-to-html branch April 6, 2023 16:07
bitspittle added a commit to varabyte/kobweb that referenced this pull request Apr 14, 2023
igordmn added a commit to JetBrains/compose-multiplatform-core that referenced this pull request Jan 3, 2024
Fixes https://youtrack.jetbrains.com/issue/COMPOSE-711/Support-collectioncollection

- ignore jvmMain/jvmTest implementation from AOSP
- use atomicfu. it isn't stable, but we already use it in `runtime`.

We will publish collections as org.jetbrains.compose.collection-internal:$composeVersion.

If in the future we decide to change the group or decouple versioninig from Compose, we can use Maven group relocation (example JetBrains/compose-multiplatform#2997)
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