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.

Sign up for GitHub

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

Extend core ideas documentation #3405

Merged
merged 3 commits into from
Feb 18, 2024
Merged

Extend core ideas documentation #3405

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
575344d
Extend core ideas documentation
qwerty287 Feb 18, 2024
63bedc3
link
qwerty287 Feb 18, 2024
f196de9
Apply suggestions from code review
qwerty287 Feb 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 0 additions & 7 deletions docs/docs/92-development/01-getting-started.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,12 +1,5 @@
# Getting started

## Core ideas

- A (e.g. pipeline) configuration should never be [turing complete](https://en.wikipedia.org/wiki/Turing_completeness) (We have agents to exec things 🙂).
- If possible follow the [KISS principle](https://en.wikipedia.org/wiki/KISS_principle).
- What is used most should be default.
- Keep different topics separated, so you can write plugins, port new ideas ... more easily, see [Architecture](./05-architecture.md).

You can develop on your local computer by following the [steps below](#preparation-for-local-development) or you can start with a fully prepared online setup using [Gitpod](https://github.com/gitpod-io/gitpod) and [Gitea](https://github.com/go-gitea/gitea).

## Gitpod
Expand Down
26 changes: 26 additions & 0 deletions docs/docs/92-development/02-core-ideas.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
# Core ideas

- A (e.g. pipeline) configuration should never be [turing complete](https://en.wikipedia.org/wiki/Turing_completeness) (We have agents to exec things 🙂).
qwerty287 marked this conversation as resolved.
Show resolved Hide resolved
- If possible, follow the [KISS principle](https://en.wikipedia.org/wiki/KISS_principle).
- What is used most should be default.
qwerty287 marked this conversation as resolved.
Show resolved Hide resolved
- Keep different topics separated, so you can write plugins, port new ideas ... more easily, see [Architecture](./05-architecture.md).

## Addons and extensions

If you wonder whether your contribution will be accepted to be merged into our core or it's better to write it as an
[addon](../30-administration/75-addons/00-overview.md), [extension](../30-administration/100-external-configuration-api.md) or an
[external custom backend](../30-administration/22-backends/50-custom-backends.md), please check these points:
qwerty287 marked this conversation as resolved.
Show resolved Hide resolved

- Is your change very specific to your setup and won't likely be used by anybody else?
- Does your change violate the [guidelines](#guidelines)?
qwerty287 marked this conversation as resolved.
Show resolved Hide resolved

Both should be false if you open a pull request to get your change into the core repository.
qwerty287 marked this conversation as resolved.
Show resolved Hide resolved

### Guidelines

#### Forges

A new forge must support these features:

- OAuth2
- Webhooks