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

Maintain app #208

Closed
1 task done
mmcint opened this issue Jan 16, 2024 · 4 comments
Closed
1 task done

Maintain app #208

mmcint opened this issue Jan 16, 2024 · 4 comments

Comments

@mmcint
Copy link
Contributor

mmcint commented Jan 16, 2024

Contact Details

[email protected]

Why

I enjoy this list and frequently use it as reference. I’m relatively new to SWUNG and want to get more involved. I also want to learn maintainer skills.

How have you participated

I’m relatively new to swung (about 1yr). I participate in discussions on the slack/Mattermost channels. I frequently use the resources on this repo for my daily work as a geo data scientist @ThinkOnward (formerly Studio X).

I’m a geoscientist by education, with a BS and MS from Kansas State. I worked as a geophysicist at APC/OXY from 2014-2021 in the GoM. I left to pursue an opportunity in data science.

Code of Conduct

  • I agree to follow this project's Code of Conduct
@mmcint
Copy link
Contributor Author

mmcint commented Jan 16, 2024

@JustinGOSSES @leouieda

@JustinGOSSES
Copy link
Contributor

Sweet, welcome @mmcint . I added you to the awesome team, so you should have permissions to approve PRs.

There’s a few open PRs. I’m going to suggest an order to get a few experiences out of the way for you as a new maintainer.

  1. Approve the DOO badge PR first. I suspect you can approve it without any merge conflicts added DOI badge #205
  2. Approving that PR will probably make others in que out of date, which will give you a chance to fix the conflicts. Your PR Update Seismic tools section in README.md with Synthoseis package #209 will likely say needs updating after step 1. Makes changes and then I’ll approve & merge. We try to only approve our own PRs in cases where debugging an issue.
  3. Last, update someone else’s PR and confirm their content is awesome with the remaining PR Propose gstlearn to Awesome list #207

@JustinGOSSES
Copy link
Contributor

If you run into problems or have questions, you can use this issue or Mattermost DM as makes sense for you.

@mmcint
Copy link
Contributor Author

mmcint commented Jan 16, 2024

Sweet. Thanks Justin!

@mmcint mmcint closed this as completed Jan 16, 2024
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

No branches or pull requests

2 participants