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

Switch to TeamCity and Qodana #822

Closed
wants to merge 75 commits into from
Closed

Switch to TeamCity and Qodana #822

wants to merge 75 commits into from

Conversation

DRSchlaubi
Copy link
Member

No description provided.

@DRSchlaubi DRSchlaubi changed the title Feature/teamcity Switch to TeamCity and Qodana Apr 25, 2023
@lukellmann
Copy link
Member

can you merge 0.9.x into this first? will be easier to review because some changes are already on 0.9.x

[email protected] and others added 26 commits April 25, 2023 20:47
- Remove no longer needed patch
- Add GraalVM CI skeleton
- Add DocsCI
- Update CLI command in README.md
@lukellmann
Copy link
Member

lukellmann commented Apr 25, 2023

can you merge 0.9.x into this first? will be easier to review because some changes are already on 0.9.x

thanks

gradle/libs.versions.toml Outdated Show resolved Hide resolved
common/build.gradle.kts Outdated Show resolved Hide resolved
qodana.yaml Outdated Show resolved Hide resolved
@@ -5,5 +5,7 @@ out/

**/build/*
!**/build/generated/
**/build/generated/**/test
Copy link
Member

Choose a reason for hiding this comment

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

where do these generated files come from?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think this came from the Graal PR to prevent commiting the reflect-config.json for tests

Copy link
Member

Choose a reason for hiding this comment

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

can we keep graal stuff out of this pr? i think we should

@lukellmann lukellmann mentioned this pull request Apr 27, 2023
@lukellmann
Copy link
Member

there are merge conflicts now

# Conflicts:
#	.github/workflows/deployment-ci.yml
#	.github/workflows/docs-ci.yml
#	build.gradle.kts
#	buildSrc/src/main/kotlin/Projects.kt
#	buildSrc/src/main/kotlin/kord-module.gradle.kts
#	buildSrc/src/main/kotlin/kord-publishing.gradle.kts
#	common/build.gradle.kts
@lukellmann
Copy link
Member

there are merge conflicts now

resolved them

@lukellmann lukellmann changed the base branch from 0.9.x to main June 17, 2023 13:53
@DRSchlaubi DRSchlaubi mentioned this pull request Aug 5, 2023
9 tasks
@DRSchlaubi
Copy link
Member Author

Replaced by #855 and kordlib/.teamcity

@DRSchlaubi DRSchlaubi closed this Jan 30, 2024
@DRSchlaubi DRSchlaubi deleted the feature/teamcity branch March 10, 2024 15:30
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