Skip to content

Feature/xamarin#16

Open
MaximeEglem wants to merge 33 commits into
DarqueWarrior:masterfrom
MaximeEglem:feature/xamarin
Open

Feature/xamarin#16
MaximeEglem wants to merge 33 commits into
DarqueWarrior:masterfrom
MaximeEglem:feature/xamarin

Conversation

@MaximeEglem

Copy link
Copy Markdown
Contributor

Hello Donovan,

I would like to participate to your project by adding Xamarin environments (Xamarin Forms and Xamarin native).

What has been done as of now:

Xamarin Forms project or Xamarin.iOS and Xamarin.Android ready to be committed.

Build creation:

  • Tasks that allow you to update your Android and iOS version (Thanks James Montemagno)
  • Tasks that build Xamarin.iOS
  • Tasks that build Xamarin.Android
  • Creation of artifacts ready for the release

Two releases :

  • One for iOS : Send to App Center
  • One for Android : Send to App Center

@DarqueWarrior

Copy link
Copy Markdown
Owner

Thank you so much. I will start reviewing this week.

@DarqueWarrior

DarqueWarrior commented Mar 27, 2018

Copy link
Copy Markdown
Owner

It appears you are using an extension for some of your tasks. You need to check that the tasks exist in the account and if not error and tell the user to install them. I used to do this with the docker task.

@DarqueWarrior

Copy link
Copy Markdown
Owner

I installed James's extension and got further but I am still getting errors. One question why two pipelines? Why not combine them?

@DarqueWarrior

DarqueWarrior commented Mar 27, 2018

Copy link
Copy Markdown
Owner

Where is the Resign IPA file task? I can't seem to find it so my iOS pipeline fails.
x Tasks with versions 'Resign IPA file:1.*' are not valid for deploy phase 'Run on agent' in environment Dev.

@MaximeEglem

Copy link
Copy Markdown
Contributor Author

I used to split the pipeline for errors. But tell me, if you want one, I will change it.

@MaximeEglem

Copy link
Copy Markdown
Contributor Author

I am gonna check your docker task to implement it, I didn't know that you already have a task that check extensions.

… tell the user to install them.

- Check Extension Mobile App Tasks for iOS and Android (by James Montemagno)
- Check Extension Apple App Store (by Microsoft)
@DarqueWarrior

Copy link
Copy Markdown
Owner

I had to push some changes for a demo I am giving next week. Can you rebase to fix conflicts?

@MaximeEglem

MaximeEglem commented Apr 18, 2018

Copy link
Copy Markdown
Contributor Author

Hey, don’t hesitate to tell me the next steps if you had the chance to test it a bit more.

@DarqueWarrior

Copy link
Copy Markdown
Owner

So sorry for the delay we are all heads down leading into Build next week. I promise I will get back to this ASAP.

@DarqueWarrior

Copy link
Copy Markdown
Owner

I have not forgotten about you.

The hosted build agents are broken at the moment so all my integration tests are failing once they are patched I will return to this.

@MaximeEglem

Copy link
Copy Markdown
Contributor Author

Hello Donovan, thx for your reply.

I merged my branch with your master to avoid conflict. I also merge the two pipelines (iOS and Android). However I need to add again the good tasks. I will do that this week.
There is still some work to do. But don't hesitate to tell me what you thing after testing.

@DarqueWarrior

Copy link
Copy Markdown
Owner

My plate is clearing so I can finally get to this.

@MaximeEglem

Copy link
Copy Markdown
Contributor Author

Good news :), just know that I'm still here.

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