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.

Sign up for GitHub

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

Jump to bottom

Finish Kotlin migration #1012

Merged
merged 103 commits into from
Nov 20, 2024
Merged

Finish Kotlin migration #1012

merged 103 commits into from
Nov 20, 2024

Conversation

Goooler
Copy link
Member

@Goooler Goooler commented Nov 5, 2024 •
edited
Loading

Closes #980.

  • CHANGELOG's "Unreleased" section has been updated, if applicable.

@Goooler Goooler changed the title Migrate to kotlin Migrate to Kotlin Nov 5, 2024
@Goooler Goooler mentioned this pull request Nov 5, 2024
Closed
1 task
@Goooler Goooler force-pushed the migrate-to-kotlin branch 5 times, most recently from 4156657 to 5c441da Compare November 5, 2024 05:58
@Goooler Goooler force-pushed the migrate-to-kotlin branch 2 times, most recently from 7b18a3a to 4dd1793 Compare November 7, 2024 02:32
@Goooler
Copy link
Member Author

Goooler commented Nov 18, 2024 

~/IdeaProjects/shadow/src/main/kotlin git:[main]
cloc .
      45 text files.
      45 unique files.                              
       0 files ignored.

github.com/AlDanial/cloc v 2.02  T=0.02 s (2730.9 files/s, 206092.2 lines/s)
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
Kotlin                          45            442            348           2606
-------------------------------------------------------------------------------
SUM:                            45            442            348           2606
-------------------------------------------------------------------------------

@Goooler
Copy link
Member Author

Goooler commented Nov 18, 2024

This was referenced Nov 18, 2024
Merged
Merged
Merged
This was referenced Nov 18, 2024
Merged
Merged
Merged
@Goooler Goooler mentioned this pull request Nov 19, 2024
Merged
@Goooler Goooler mentioned this pull request Nov 20, 2024
Merged
This was referenced Nov 20, 2024
Merged
Merged
Merged
Merged
Merged
@Goooler
Merge remote-tracking branch 'origin/main' into migrate-to-kotlin
7bdc4ab 
// Conflicts:
//	api/src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/legacy/LegacyShadowPlugin.kt
//	src/main/groovy/com/github/jengelman/gradle/plugins/shadow/legacy/LegacyShadowPlugin.groovy
//	src/main/kotlin/com/github/jengelman/gradle/plugins/shadow/legacy/LegacyShadowPlugin.kt
@Goooler
Fix merge
672f73a
@Goooler Goooler changed the title Migrate to Kotlin Finish Kotlin migration Nov 20, 2024
@Goooler Goooler enabled auto-merge (squash) November 20, 2024 15:33
@Goooler Goooler merged commit 976f589 into main Nov 20, 2024
9 checks passed
@Goooler Goooler deleted the migrate-to-kotlin branch November 20, 2024 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot Copilot left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Rewrite this plugin in Kotlin
1 participant
@Goooler