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

Update CODEOWNERS to facilitate x-team collaboration #267

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

jhkennedy
Copy link
Contributor

Tools, SciDev, and Services will want to collaborate on this repository since each team will be deploying HyP3 for users with this plugin.

Notably, I do not expect or want to require approvals from both teams; this is just to facilitate communication. One approval from anyone on either of these three teams should generally be sufficient.

Parent tracking issue is here: https://github.com/ASFHyP3/.github-private/issues/7

@jhkennedy jhkennedy requested a review from a team as a code owner February 6, 2025 00:07
@jhkennedy jhkennedy added the bumpless Changes to documentation, CI/CD pipelines, etc that don't affect the project's version label Feb 6, 2025
@jhkennedy jhkennedy requested review from a team February 6, 2025 00:07
Copy link
Contributor

github-actions bot commented Feb 6, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
/home/runner/micromamba/envs/hyp3-isce2/lib/python3.11/site-packages/hyp3_isce2
   __main__.py26292%41, 55
   burst.py2635778%95, 105, 118–121, 134–139, 153–166, 184–191, 209–219, 287, 335–343, 363–377, 597–599, 625
   dem.py53394%103–112
   insar_stripmap.py755033%37–100, 104–106, 120–137
   insar_tops.py825533%52–117, 147–200, 225–243
   insar_tops_burst.py1359827%66–152, 163–217, 229–251, 255–257, 288–320
   logger.py4175%9
   merge_tops_bursts.py6319685%186, 203, 272, 278, 335, 351, 359, 394, 398, 486, 513, 591, 631–654, 702–717, 741, 746, 805, 834, 978, 1161–1181, 1200–1217, 1240–1273, 1284–1292, 1302–1322, 1326
   packaging.py19914826%45–47, 73–97, 145–243, 262–278, 297–307, 319–347, 359–361, 390–479, 483–489
   s1_auxcal.py24292%55, 63
   slc.py492353%23–30, 34–39, 50–62
   stripmapapp_alos.py50786%103, 144, 147–152
   topsapp.py71889%79, 109, 131–137
   utils.py1921095%108–119, 124, 202, 224–226, 233, 390, 435
   water_mask.py59592%94, 106–109
TOTAL193456571% 

Tests Skipped Failures Errors Time
81 0 💤 0 ❌ 0 🔥 35.301s ⏱️

@jtherrmann
Copy link
Contributor

Ignore mypy (new errors due to recent upgrade)

@jhkennedy jhkennedy merged commit eb38e21 into develop Feb 6, 2025
11 of 13 checks passed
@jhkennedy jhkennedy deleted the jhkennedy-patch-1 branch February 6, 2025 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bumpless Changes to documentation, CI/CD pipelines, etc that don't affect the project's version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants