Skip to content

feat: 🎸 [HCPSDKFIORIUIKIT-2680] Migrate What's New component #1206

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

JuliaSchnSAP
Copy link
Collaborator

No description provided.

@JuliaSchnSAP JuliaSchnSAP requested a review from a team as a code owner August 7, 2025 01:28
@JuliaSchnSAP JuliaSchnSAP requested review from billzhou0223 and removed request for a team August 7, 2025 01:28
/// producing an optional view that is visible only when the condition
/// evaluates to `true`.
public static func buildIf(_ content: (some View)?) -> some WhatsNewPageList {
SingleWhatsNewPage(view: content == nil ? AnyView(EmptyView()) : AnyView(content!))
Copy link

Choose a reason for hiding this comment

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

Force Unwrapping Violation: Force unwrapping should be avoided. (force_unwrapping)

WhatsNewPagePair(first: c0, remainder: WhatsNewPagePair(first: c1, remainder: WhatsNewPagePair(first: c2, remainder: WhatsNewPagePair(first: c3, remainder: WhatsNewPagePair(first: c4, remainder: WhatsNewPagePair(first: c5, remainder: WhatsNewPagePair(first: c6, remainder: WhatsNewPagePair(first: c7, remainder: SingleWhatsNewPage(view: c8)))))))))
}

public static func buildBlock(_ c0: some View, _ c1: some View, _ c2: some View, _ c3: some View, _ c4: some View, _ c5: some View, _ c6: some View, _ c7: some View, _ c8: some View, _ c9: some View) -> some WhatsNewPageList {
Copy link

Choose a reason for hiding this comment

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

Function Parameter Count Violation: Function should have 5 parameters or less: it currently has 10 (function_parameter_count)
Missing Docs Violation: public declarations should be documented. (missing_docs)

WhatsNewPagePair(first: c0, remainder: WhatsNewPagePair(first: c1, remainder: WhatsNewPagePair(first: c2, remainder: WhatsNewPagePair(first: c3, remainder: WhatsNewPagePair(first: c4, remainder: WhatsNewPagePair(first: c5, remainder: WhatsNewPagePair(first: c6, remainder: SingleWhatsNewPage(view: c7))))))))
}

public static func buildBlock(_ c0: some View, _ c1: some View, _ c2: some View, _ c3: some View, _ c4: some View, _ c5: some View, _ c6: some View, _ c7: some View, _ c8: some View) -> some WhatsNewPageList {
Copy link

Choose a reason for hiding this comment

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

Function Parameter Count Violation: Function should have 5 parameters or less: it currently has 9 (function_parameter_count)
Missing Docs Violation: public declarations should be documented. (missing_docs)

WhatsNewPagePair(first: c0, remainder: WhatsNewPagePair(first: c1, remainder: WhatsNewPagePair(first: c2, remainder: WhatsNewPagePair(first: c3, remainder: WhatsNewPagePair(first: c4, remainder: WhatsNewPagePair(first: c5, remainder: SingleWhatsNewPage(view: c6)))))))
}

public static func buildBlock(_ c0: some View, _ c1: some View, _ c2: some View, _ c3: some View, _ c4: some View, _ c5: some View, _ c6: some View, _ c7: some View) -> some WhatsNewPageList {
Copy link

Choose a reason for hiding this comment

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

Function Parameter Count Violation: Function should have 5 parameters or less: it currently has 8 (function_parameter_count)
Missing Docs Violation: public declarations should be documented. (missing_docs)

WhatsNewPagePair(first: c0, remainder: WhatsNewPagePair(first: c1, remainder: WhatsNewPagePair(first: c2, remainder: WhatsNewPagePair(first: c3, remainder: WhatsNewPagePair(first: c4, remainder: SingleWhatsNewPage(view: c5))))))
}

public static func buildBlock(_ c0: some View, _ c1: some View, _ c2: some View, _ c3: some View, _ c4: some View, _ c5: some View, _ c6: some View) -> some WhatsNewPageList {
Copy link

Choose a reason for hiding this comment

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

Function Parameter Count Violation: Function should have 5 parameters or less: it currently has 7 (function_parameter_count)
Missing Docs Violation: public declarations should be documented. (missing_docs)

WhatsNewPagePair(first: c0, remainder: WhatsNewPagePair(first: c1, remainder: WhatsNewPagePair(first: c2, remainder: WhatsNewPagePair(first: c3, remainder: SingleWhatsNewPage(view: c4)))))
}

public static func buildBlock(_ c0: some View, _ c1: some View, _ c2: some View, _ c3: some View, _ c4: some View, _ c5: some View) -> some WhatsNewPageList {
Copy link

Choose a reason for hiding this comment

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

Function Parameter Count Violation: Function should have 5 parameters or less: it currently has 6 (function_parameter_count)
Missing Docs Violation: public declarations should be documented. (missing_docs)


public extension WhatsNewPageList {
var isEmpty: Bool {
count == 0
Copy link

Choose a reason for hiding this comment

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

Empty Count Violation: Prefer checking isEmpty over comparing count to zero. (empty_count)

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