Skip to content

Code organization improvements#3

Open
alexandarZ wants to merge 2 commits intobrills:mainfrom
alexandarZ:main
Open

Code organization improvements#3
alexandarZ wants to merge 2 commits intobrills:mainfrom
alexandarZ:main

Conversation

@alexandarZ
Copy link

@alexandarZ alexandarZ commented Aug 10, 2024

Project refactoring for further improvements

Hello!
First, I want to express my gratitude for creating such an amazing project with SatHunter. Your work is truly inspiring, and I deeply appreciate the effort you’ve put into it.

As a first step towards better project organization and to pave the way for new contributions, I’ve made some changes to the project structure and refactored some variable names. I hope that these changes will make it easier for others to navigate the project and contribute in the future, as I will do.

I hope you find these updates helpful, and I look forward to seeing the project grow and evolve.

Thank you again for your hard work and dedication!

Best regards,
Alex, YT3TCA

@alexandarZ alexandarZ marked this pull request as draft August 11, 2024 13:43
@alexandarZ alexandarZ changed the title Small readme improvements Code organization improvements Aug 11, 2024
@alexandarZ alexandarZ marked this pull request as ready for review August 11, 2024 13:47
@brills
Copy link
Owner

brills commented Aug 23, 2024

Hey Aleksandar -- thanks for the contribution, and sorry for the delayed reply. This will take a bit time for me to review but I do want to thank you first for the efforts to make the project cleaner.

// For iOS, using UIKit
let userInterfaceStyle = UITraitCollection.current.userInterfaceStyle
return userInterfaceStyle == .dark ? .dark : .light
#elseif os(macOS)
Copy link
Owner

Choose a reason for hiding this comment

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

just curious -- is this supposed to be the way to support both macOS and iOS?

import Foundation
import OSLog

fileprivate let kPi = 3.1415926535897932384626433832795028841415926
Copy link
Owner

Choose a reason for hiding this comment

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

ouch. I wonder if any library provides this constant?

Double.pi?

if !requestNew {
return uuid
}
if let stored = UserDefaults.standard.string(forKey: "BTID") {
Copy link
Owner

Choose a reason for hiding this comment

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

some files are formatted with 4-space indentation while some others are with 2-space...

I remember using SwiftFormat and did not bother tweaking any its settings. Which formatter do you use and which preset ?

Copy link
Owner

@brills brills left a comment

Choose a reason for hiding this comment

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

That's all my comments. Thank you again.

Copy link
Owner

Choose a reason for hiding this comment

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

Should we update the Changelog / Release note about this?

)
Spacer()
Text("\(String(format: "%.0f", maxEl!)) deg")
Text("\(String(format: "%.0f", maxEl!)) °")
Copy link
Owner

Choose a reason for hiding this comment

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

(no action required) thank you!

Spacer()
Text(userGrid)
}
HStack {
Copy link
Owner

Choose a reason for hiding this comment

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

is there a better way to indicate this? The decision of presenting local time is only for my convenience and I'm not sure if it's a surprise to other users.

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