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

feat: migrate project from GitHub Enterprise #2

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

robert-mitchell-iii
Copy link
Contributor

@robert-mitchell-iii robert-mitchell-iii commented Feb 14, 2025

Issue

#1

Summary

  • This PR migrates the project from GitHub enterprise to github.com.

Changes

  • Copy code over from GHE.
  • Update CI workflow(s) to run on GitHub-hosted runners.
  • Configure CODEOWNERS file.
  • Update PR template to reference this project's issues.

Notes / References

  • Link to this repo was added to the internal project added.
  • Fuji-san was added a co-author to the migration commit.

Checklist

  • Link to issue specified
  • Implementation satisfies the stated objective(s)
  • Test(s) added as appropriate
  • Documentation created/updated

@robert-mitchell-iii robert-mitchell-iii self-assigned this Feb 14, 2025
Copy link

Code Coverage

File Coverage [98.61%]
src/main/kotlin/jp/co/lycorp/geojson/BBox.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/Edge.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/Feature.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/FeatureCollection.kt 88.89%
src/main/kotlin/jp/co/lycorp/geojson/FeatureId.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/GeoJsonObject.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/Geometry.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/GeometryCollection.kt 75.00%
src/main/kotlin/jp/co/lycorp/geojson/LineString.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/LinearRing.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/MultiLineString.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/MultiPoint.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/MultiPolygon.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/Point.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/Polygon.kt 94.44%
src/main/kotlin/jp/co/lycorp/geojson/Position.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/Surface.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/algorithm/GeometryHelper.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/jackson/BBoxDeserializer.kt 96.55%
src/main/kotlin/jp/co/lycorp/geojson/jackson/BBoxSerializer.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/jackson/FeatureIdDeserializer.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/jackson/FeatureIdSerializer.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/jackson/PositionDeserializer.kt 95.65%
src/main/kotlin/jp/co/lycorp/geojson/jackson/PositionSerializer.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/validator/LineStringValidator.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/validator/LinearRingValidator.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/validator/PolygonValidator.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/validator/PositionValidator.kt 100.00%
src/main/kotlin/jp/co/lycorp/geojson/validator/RingIntersectionValidator.kt 100.00%
Total Project Coverage 98.61%

test:
name: Test
runs-on:
- ubuntu-latest
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -0,0 +1 @@
* @robert-mitchell-iii @masato-noda @akira-kawakatsu
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added each of our account names to the code owners list for now, but we could look into creating a team in the future.

Copy link

@masato-noda masato-noda left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@akira-kawakatsu akira-kawakatsu left a comment

Choose a reason for hiding this comment

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

LGTM

@robert-mitchell-iii robert-mitchell-iii merged commit a6fcf3b into main Feb 26, 2025
2 checks passed
@robert-mitchell-iii robert-mitchell-iii deleted the feature/migrate-from-ghe branch March 6, 2025 04:52
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