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

indent #6726

Merged
merged 2 commits into from Jan 26, 2023
Merged

indent #6726

merged 2 commits into from Jan 26, 2023

Conversation

ryuusama09
Copy link
Contributor

Thank you for contributing to SunPy

🚨 IMPORTANT 🚨

We know that working on code and submitting pull requests takes effort, and we appreciate your time.
Thank you.

Please be aware that everyone has to follow our code of conduct

Furthermore, you might need to check with your work place if you are allowed to contribute code

  • Please try to create an issue before creating a Pull Request.
  • Please use the following Git commit message style
    • Use the future tense ("Adds feature" not "Added feature")
    • Limit the first line to 72 characters or less

We have several automatic features that run when a pull request is open.
They can appear daunting but do not worry about them!
This a brief explanation of them.

β†‘πŸ‘† DELETE above before submitting πŸ‘†β†‘

PR Description

fixed indentation and made adequate spacing.
fixes #6412

@ryuusama09 ryuusama09 requested a review from a team as a code owner January 25, 2023 12:26
@ryuusama09
Copy link
Contributor Author

pre-commit.ci autofix

@nabobalis nabobalis added timeseries Affects the timeseries submodule No Changelog Entry Needed No Backport A PR that isn't to be backported to any release branch. (To be used as a flag to other maintainers) labels Jan 25, 2023
@nabobalis
Copy link
Contributor

What does it look like before and after these changes?

@ryuusama09
Copy link
Contributor Author

SunPy TimeSeries

Observatory: GOES-15
Instrument: X-ray Detector
Channel(s): xrsa
xrsb
Start Date: 2011-06-07 00:00:00
End Date: 2011-06-07 23:59:58
Center Date: 2011-06-07 11:59:58
Resolution: 2.048 s
Samples per Channel: 42177
Data Range(s): xrsa 3.64E-06
xrsb 2.54E-05
Units: W / m2
this is what is looked previously as mentioned in the issue.

What does it look like before and after these changes?

i just tested the indentation without using an actual example and introduced my own variables and worked fine . Do you want to see that ?

@nabobalis
Copy link
Contributor

i just tested the indentation without using an actual example and introduced my own variables and worked fine . Do you want to see that ?

Yes please

@ryuusama09
Copy link
Contributor Author

i just tested the indentation without using an actual example and introduced my own variables and worked fine . Do you want to see that ?

Yes please

image
here

@nabobalis
Copy link
Contributor

Looks good

@ryuusama09
Copy link
Contributor Author

Looks good

thank you !

@nabobalis nabobalis merged commit c896f4f into sunpy:main Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No Backport A PR that isn't to be backported to any release branch. (To be used as a flag to other maintainers) No Changelog Entry Needed timeseries Affects the timeseries submodule
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve tabulation in TimeSeries repr
2 participants