Skip to content

Feature/add more instruments - #7

Open
OfTheWolf wants to merge 2 commits into
AudioKit:mainfrom
OfTheWolf:feature/add-more-instruments
Open

OfTheWolf wants to merge 2 commits into
AudioKit:mainfrom
OfTheWolf:feature/add-more-instruments

Conversation

@OfTheWolf

Copy link
Copy Markdown

Added the implementations of blowHole, brass, plucked string, sax, sitar, stifkarp (karplus/strong+) instruments and created a new demo project for them.


var body: some View {
VStack(spacing: 20) {
List{

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Opening Brace Spacing Violation: Opening braces should be preceded by a single space and on the same line as the declaration. (opening_brace)

import Foundation

enum Instrument: String, Hashable, CaseIterable, Identifiable {
var id: Self { self }

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Identifier Name Violation: Variable name should be between 3 and 40 characters long: 'id' (identifier_name)

#if os(iOS)
do {
try AVAudioSession.sharedInstance().setCategory(.playAndRecord,
options: [.defaultToSpeaker, .mixWithOthers, .allowBluetoothA2DP])

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Line Length Violation: Line should be 120 characters or less: currently 130 characters (line_length)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant